Request for review (JVMTI spec, small)

[Daniel D. Daugherty]

I think that reference to 5.3.1 is correct. The previous line:

    Removed incorrect statement in GetClassloaderClasses

tends to indicate that this was a class loader issue. So the
reference to the BootstapClassLoader is probably right.

Dan



On 5/5/2011 8:50 AM, Keith McGuigan wrote:
>
>>
>> This one seems to refer to the wrong Chapter/Section - Constant pool 
>> should be 4.4
>
> Actually hold on a sec:  The original URL: 
> http://java.sun.com/docs/books/vmspec/2nd-edition/html/ConstantPool.doc.html#79383 
>
> Does refer to chapter 5.3.1.  Do you think the original URL is wrong?  
> I don't know what "line" the comment is referring to so I'll likely 
> just leave this as it is and it can be clarified later.
>
> -- 
> - Keith
>
>> *** 13839,13849 ****
>>        Clarify that stack frames are JVM Spec frames.
>>        Split can_get_source_info into can_get_source_file_name, 
>> can_get_line_numbers,
>>        and can_get_source_debug_extension.
>>        PopFrame cannot have a native calling method.
>>        Removed incorrect statement in GetClassloaderClasses
>> !       (see 
>> http://java.sun.com/docs/books/vmspec/2nd-edition/html/ConstantPool.doc.html#79383). 
>>
>>    </change>
>>    <change date="24 July 2003" version="v79">
>>        XML and text fixes.
>>        Move stack frame description into Stack Frame category.
>>    </change>
>> --- 13807,13817 ----
>>        Clarify that stack frames are JVM Spec frames.
>>        Split can_get_source_info into can_get_source_file_name, 
>> can_get_line_numbers,
>>        and can_get_source_debug_extension.
>>        PopFrame cannot have a native calling method.
>>        Removed incorrect statement in GetClassloaderClasses
>> !       (see <vmspec chapter="5.3.1"/>).
>>
>> Otherwise seems fine - though I don't claim to be fluent in xml :)
>>
>> David
>