RFR: 8002070 Remove the stack search for a resource bundle for Logger to use
Jim Gish
jim.gish at oracle.com
Tue Mar 5 22:29:41 UTC 2013
The webrev has been updated as requested:
http://cr.openjdk.java.net/~jgish/Bug8002070-RemoveResourceBundleStackSearch/
A CCC request has been filed.
Thanks,
Jim
On 03/05/2013 02:12 PM, Jim Gish wrote:
>
> On 03/03/2013 09:15 AM, Alan Bateman wrote:
>> On 01/03/2013 21:25, Jim Gish wrote:
>>> Please review
>>> http://cr.openjdk.java.net/~jgish/Bug8002070-RemoveResourceBundleStackSearch/
>>> <http://cr.openjdk.java.net/%7Ejgish/Bug8002070-RemoveResourceBundleStackSearch/>
>>>
>>>
>>> This removes the stack search from Logger.findResourceBundle()
>>>
>>> Note that because this is a change in the spec/functionality, a CCC
>>> request will be filed.
>> It's good to have this removed. The changes to the caching seem okay,
>> at least I don't see anything obviously wrong.
>>
>> One suggestion is to change "SystemClassLoader" into system
>> ClassLoader and link it to the ClassLoader.getSystemClassLoader.
> done
>>
>> For the LoggerResourceBundleRace test then does it have to run in its
>> own VM?
> Because we no longer search up the stack for the bundles, this test
> fails unless run in its own vm. However, to avoid this, I'll try to
> change the test to set the context classloader.
>>
>> -Alan.
>
--
Jim Gish | Consulting Member of Technical Staff | +1.781.442.0304
Oracle Java Platform Group | Core Libraries Team
35 Network Drive
Burlington, MA 01803
jim.gish at oracle.com
More information about the core-libs-dev
mailing list