On Thu, 6 Aug 2026 12:01:09 GMT, Coleen Phillimore <[email protected]> wrote:

>> It is not why we kept the method Symbol around, that was for a special case 
>> and cost both footprint, gc time and performance.  If the format of the name 
>> might be a problem, I could use redefineDeleted or something like that.  I 
>> don't know why `<redefined deleted>` would be since the method name could 
>> also be `<init>`.  I could also rename and use this symbol but it doesn't 
>> really describe why the method name can't be found.
>> 
>>       /* used by ClassFormatError when class name is not known yet */        
>>                                          \
>>       template(unknown_class_name,                        "<Unknown>")
>
> Now I remember, we used to return nullptr for name which caused some old tool 
> to crash, so we shouldn't return nullptr.

Unless the name is `<init>` or `<clinit>`, if it contains `<` or `>` it is an 
illegal name.  Symbols should only contain valid language elements. (Yes I'm 
being a bit picky - sorry.)

If the reason for not having the actual name is important to convey then 
perhaps something more explicit like `Unknown_deleted_by_redefinition` would 
work?

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/32202#discussion_r3733238502

Reply via email to