On Tue, 4 Nov 2025 10:49:31 GMT, Johan Sjölen <[email protected]> wrote:
>> src/jdk.hotspot.agent/share/classes/sun/jvm/hotspot/oops/ConstantPool.java >> line 482: >> >>> 480: int basePos = offs.at(bsmIndex); >>> 481: int argv = basePos + INDY_ARGV_OFFSET; >>> 482: int argc = getBootstrapMethodArgsCount(bsmIndex); >> >> Nit: Consider to make it shorter: >> `getBootstrapMethods` => `getBSMs` >> `getBootstrapMethodArgsCount` => `getBSMArgsCount` > > I'm keeping it verbose, as that's the general style of this file. Yes please I like the full names if possible rather than abbreviations. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/27198#discussion_r2490567923
