[9] RFR: 8171139: Simplify ResourceBundle.CacheKey and ClassLoader may not be needed
Naoto Sato
naoto.sato at oracle.com
Thu Jan 12 21:20:11 UTC 2017
Realized moduleRef & callerRef would never be null with this new
version. Thus modified the patch as:
http://cr.openjdk.java.net/~naoto/8171139/webrev.03/
Naoto
On 1/12/17 11:16 AM, Naoto Sato wrote:
> Thank you, Mandy and Daniel for your comments. I incorporated your
> suggestions and updated the fix as follows:
>
> http://cr.openjdk.java.net/~naoto/8171139/webrev.02/
> http://cr.openjdk.java.net/~naoto/8171140/webrev.00/
>
> Please review.
>
> Naoto
>
>
> On 1/11/17 5:45 PM, Naoto Sato wrote:
>> Decided to include the fix to 8171140 [1] as well, as they are closely
>> related. Here is the updated webrev.
>>
>> http://cr.openjdk.java.net/~naoto/8171139.8171140/webrev.01/
>>
>> Additional changes to the version 00 are to mainly remove
>> clearCache(Module) method, as clearCache() is chiefly used in
>> combination with ResourceBundle.Control, which is not supported in named
>> modules. Still named modules can issue clearCache() with no argument
>> which will result in the same effect. Also, other clearCache() overloads
>> have clearer method descriptions.
>>
>> CCC for 8171140 will be filed accordingly.
>>
>> Naoto
>> --
>> [1]: https://bugs.openjdk.java.net/browse/JDK-8171140
>>
>> On 1/10/17 4:10 PM, Naoto Sato wrote:
>>> Hello,
>>>
>>> Please review the changes to the subject issue:
>>>
>>> https://bugs.openjdk.java.net/browse/JDK-8171139
>>>
>>> The changes are located at:
>>>
>>> http://cr.openjdk.java.net/~naoto/8171139/webrev.00/
>>>
>>> The changeset is the same as the one in the mail thread [1]
>>> contributed by Peter Levart, plus additional clearCache() test cases.
>>>
>>> Naoto
>>>
>>> [1]
>>> http://mail.openjdk.java.net/pipermail/core-libs-dev/2017-January/045790.html
>>>
>>> <http://mail.openjdk.java.net/pipermail/core-libs-dev/2017-January/045790.html>
>>>
>>>
>>>
More information about the core-libs-dev
mailing list