On Wed, 25 Sep 2024 12:58:22 GMT, Chen Liang <li...@openjdk.org> wrote:
>> AFAICT they already use the record names, so we're not generating redundant >> code there. > > I mean the name variable in makeHiddenClassDefiner and makeClassDefiner that > leads up here. Ah.. Yeah, either way you like, but if it's an internal name (`foo/Bar`) then including internal in the variable names consistently is good I think. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/21170#discussion_r1775187937