RFR: 8328730: Convert java/awt/print/bug8023392/bug8023392.html applet test to main [v2]
Damon Nguyen
dnguyen at openjdk.org
Mon Mar 25 16:08:24 UTC 2024
On Sat, 23 Mar 2024 08:23:25 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:
>> Damon Nguyen has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Move test out of extra dir
>
> test/jdk/java/awt/print/bug8023392.java line 116:
>
>> 114: @Override
>> 115: protected void paintComponent(Graphics g) {
>> 116: SwingUtilities2.drawChars(this, g, s.toCharArray(),
>
> cant we use `Graphics.drawChars` instead in all places `SwingUtilities2.drawChars` is used?
`Graphics.drawChars` isn't a static method, which will require additional changes to this test. Is it preferred to use `Graphics` over the `SwingUtilities2` version?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/18458#discussion_r1537844651
More information about the client-libs-dev
mailing list