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