On Wed, 27 Sep 2023 21:51:38 GMT, Alex Menkov <amen...@openjdk.org> wrote:

>> test/lib/jdk/test/lib/hprof/model/Root.java line 145:
>> 
>>> 143: 
>>> 144:     public long getReferrerId() {
>>> 145:         return refererId;
>> 
>> Spelling: `referrerId`
>
> This is existing field in the class.
> There are a number (45) of misspelled "referer" in the lib, including some 
> public methods (like getReferer() in line 140).
> If you think it makes sense to fix the, I can file a separate CR for this.

Ugghh! Line 140 was not visible by default. It doesn't look good to add a new 
method called `getReferrerId` when the other usages are `referer`. Lets go for 
consistency in this PR and fix them all in a separate one. Thanks.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/15869#discussion_r1339431813

Reply via email to