On Thu, 17 Apr 2025 19:18:18 GMT, Abhishek Kumar <abhis...@openjdk.org> wrote:
>> Few more font related tests are converted from applet and open-sourced. >> >> TestGlyphVectorLayout.java doesn't behave as per the instruction and hence >> problem-listed under >> [JDK-8354987](https://bugs.openjdk.org/browse/JDK-8354987). > > Abhishek Kumar has updated the pull request incrementally with one additional > commit since the last revision: > > Problem list test moved to manual test category Changes requested by aivanov (Reviewer). test/jdk/ProblemList.txt line 796: > 794: java/awt/print/PrinterJob/ScaledText/ScaledText.java 8231226 macosx-all > 795: java/awt/print/PrinterJob/PrintTextTest.java 8148334 macosx-all > 796: java/awt/font/GlyphVector/TestGlyphVectorLayout.java 8354987 generic-all Probably, you want to add [JDK-8355048](https://bugs.openjdk.org/browse/JDK-8355048) to the list of resolved issues, since you include problem-listing in this changeset, which makes perfect sense. test/jdk/java/awt/font/TestGraphicPlacement.java line 166: > 164: attribute, > 165: i + name.length(), > 166: i + name.length() + 1); Suggestion: as.addAttribute(TextAttribute.CHAR_REPLACEMENT, attribute, i + name.length(), i + name.length() + 1); Align the wrapped parameters to match the style in lines 158–160 above? ------------- PR Review: https://git.openjdk.org/jdk/pull/24736#pullrequestreview-2779172845 PR Review Comment: https://git.openjdk.org/jdk/pull/24736#discussion_r2050966012 PR Review Comment: https://git.openjdk.org/jdk/pull/24736#discussion_r2050970157