RFR JDK-8038310: Re-examine integration of extended Charsets (original) (raw)
Xueming Shen xueming.shen at oracle.com
Thu May 28 16:41:58 UTC 2015
- Previous message: RFR JDK-8038310: Re-examine integration of extended Charsets
- Next message: RFR JDK-8038310: Re-examine integration of extended Charsets
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
On 05/28/2015 09:05 AM, Mandy Chung wrote:
On 05/28/2015 08:53 AM, Xueming Shen wrote:
http://cr.openjdk.java.net/~sherman/80695888038310/webrev This change looks okay. This code optimizes for space and assumes one single installed provider which I am not sure if it buys us much - List.toArray might be another alternative rather than expanding/shrinking the array. But just minor point.
Thanks Mandy! Given we probably are not going to have "lots" of providers installed, probably two as the max in the foreseeable future , a List is little heavy for me:-) Let's keep this "simple code" asis for now.
-Sherman
- Previous message: RFR JDK-8038310: Re-examine integration of extended Charsets
- Next message: RFR JDK-8038310: Re-examine integration of extended Charsets
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]