On Tue, 14 Mar 2023 23:29:17 GMT, Calvin Cheung <cche...@openjdk.org> wrote:

>> Matias Saavedra Silva has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   RISCV port update
>
> src/hotspot/share/interpreter/bootstrapInfo.cpp line 234:
> 
>> 232:   if (_indy_index > -1) {
>> 233:     os::snprintf_checked(what, sizeof(what), "indy#%d", _indy_index);
>> 234:   }
> 
> Since the `else` case doesn’t have braces, maybe omit the braces for this 
> case as well?

The if statements below use braces so I think it would be better to add braces 
to the else case.

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

PR: https://git.openjdk.org/jdk/pull/12778

Reply via email to