Request Review: JDK-6479237 (cl) Add support for classloader names

[David M. Lloyd]

On 10/27/2016 09:59 AM, Mandy Chung wrote:
>
>> On Oct 27, 2016, at 7:30 AM, David M. Lloyd <david.lloyd at redhat.com> wrote:
>>
>> It looks like if you have a class loader name but not a module (i.e. it's unnamed), you get output that looks like this:
>>
>>        [...]
>>        at org.jboss.as.controller//org.jboss.as.controller.AbstractControllerService.boot(AbstractControllerService.java:390)
>>        [...]
>>
>> In this case the classloader's name is "org.jboss.as.controller".  Is the double-slash "//" intended?
>>
>
> Yes.  The format is <loader>/<module>/<fully-qualified-classname> and ifthe class loader is named but module is unnamed, it will drop <module>.  This can tell if the module is named or unnamed from the output.
>
> See the javadoc in StackTraceElement::toLoaderModuleClassName.  I will double check if the example in StackTraceElement::toString includes this.

OK Thanks.  I was looking at the StackTraceElement#toString JavaDoc and 
it was not very clear if this was the expected output; maybe it's worth 
pointing out explicitly.
-- 
- DML