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