RFR: 8283013: Simplify Arguments::parse_argument() [v3]
    Ioi Lam 
    iklam at openjdk.java.net
       
    Wed Mar 30 20:39:19 UTC 2022
    
    
  
On Wed, 30 Mar 2022 15:47:23 GMT, Calvin Cheung <ccheung at openjdk.org> wrote:
>> Ioi Lam has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   moved comment
>
> src/hotspot/share/runtime/arguments.cpp line 1069:
> 
>> 1067: 
>> 1068:   size_t name_len = size_t(arg - name);
>> 1069:   JVMFlag* flag = find_jvm_flag(name, name_len);
> 
> Consider -XX:@blah
> I think the name_len could be 0 if the preceding while loop did not increment the arg pointer.
> But I think it is ok because find_jvm_flag would return NULL on a zero name_len.
> Is it correct?
The code does handle the case where `name_len==0`, but I added a new check anyway to make the code easier to understand.
-------------
PR: https://git.openjdk.java.net/jdk/pull/7916
    
    
More information about the hotspot-dev
mailing list