RFR: 8002070 Remove the stack search for a resource bundle for Logger to use

Mandy Chung mandy.chung at oracle.com
Fri Mar 8 22:51:48 UTC 2013


On 3/8/2013 2:39 PM, Jim Gish wrote:
> I've update the webrev with the suggested changes: 
> http://cr.openjdk.java.net/~jgish/Bug8002070-RemoveResourceBundleStackSearch/ 
> <http://cr.openjdk.java.net/%7Ejgish/Bug8002070-RemoveResourceBundleStackSearch/>
>

Thumbs up.  Thanks for adding the comment in the test.  I'd suggest to 
take out the statement about "stack walk search" since this may be 
confusing to others who read it as that code will no longer exist.  
Please add this bug ID to @bug.

thanks
Mandy

> Thanks,
>    Jim
>
>
> On 03/08/2013 03:41 PM, Mandy Chung wrote:
>> On 3/5/2013 11:12 AM, Jim Gish wrote:
>>>> 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. 
>>
>> I discussed with Jim.  In jtreg samevm mode, the testclasses are not 
>> in the classpath and thus the system classloader can't find the 
>> resource bundle.  Before the fix, the test finds the resource bundle 
>> using the classloader on the stack which isn't the intent.
>>
>> Mandy
>
> -- 
> 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