On Thu, 28 Mar 2024 12:12:07 GMT, Karthik P K <k...@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