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