RFR: 8328754: Fix missing @Overrides in test [v2]
Andy Goryachev
angorya at openjdk.org
Thu Mar 28 14:54:47 UTC 2024
On Thu, 28 Mar 2024 12:12:07 GMT, Karthik P K <kpk at openjdk.org> wrote:
>> Andy Goryachev has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:
>>
>> - more files
>> - Merge branch 'master' into 8328754.overrides.test
>> - 8328754: Fix missing @Overrides in test
>
> Changes looks good.
> Found few file where `@Override` can be added for `start` method.
>
> 1. `BigGlyphIDTest.java`
> 2. `INVISIBLE_GLYPH_IDTest.java`
> 3. `LoadFonts.java`
good catch, thank you @karthikpandelu !
please re-review
-------------
PR Comment: https://git.openjdk.org/jfx/pull/1427#issuecomment-2025420842
More information about the openjfx-dev
mailing list