Review request JDK-8153944: wsimport and wsgen usage messages not printed

Mandy Chung mandy.chung at oracle.com
Wed May 25 19:48:36 UTC 2016


> On May 25, 2016, at 12:45 PM, Alan Bateman <Alan.Bateman at oracle.com> wrote:
> 
> On 25/05/2016 20:29, Mandy Chung wrote:
>> wsgen and wsimport tools use the utility classes in java.xml.bind module to localize messages.  Resource bundles are encapsulated in named modules and hence utility like com.sun.istack.internal.localization.LocalizableMessageFactory.
>> 
>> The owner of the resource bundle should be the one loading ResourceBundle.  The solution to this type of issue is to update the utility classes to accept a function to return ResourceBundle rather than by base name.  Another alternative could do the localization locally within the module and this may not be applicable to JAX-WS since jdk.xml.ws is tightly coupled with java.xml.ws.
>> 
>> Webrev at:
>>   http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8153944/webrev.00
>> 
>> This patch touches quites a few files, one per supporting class to get messages from one resource bundle.  The change is straight-forward and same pattern applies to each message file.
>> 
> I don't know area but I looked at the changes and I get the gist of this. It would be good if Aleksej could review as he's the current connection to the upstream project that owns this code.
> 

I mentioned this patch to Aleksej earlier.  Aleksej - can you review this too?

Mandy




More information about the core-libs-dev mailing list