On Wed, 22 Nov 2023 07:49:07 GMT, Andrey Turbanov <aturba...@openjdk.org> wrote:

>> Phil Race has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   8320443
>
> src/java.desktop/share/classes/sun/print/RasterPrinterJob.java line 2096:
> 
>> 2094: 
>> 2095:     synchronized protected void setGraphicsConfigInfo(AffineTransform 
>> at,
>> 2096:                                             double pw, double ph) {
> 
> Let's use blessed modifiers order
> Suggestion:
> 
>     protected synchronized void setGraphicsConfigInfo(AffineTransform at,
>                                                       double pw, double ph) {

fixed

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/16773#discussion_r1414628370

Reply via email to