On Mon, 25 Mar 2024 05:58:41 GMT, Julian Waters <jwat...@openjdk.org> wrote:

>> src/java.desktop/windows/native/libawt/windows/awt_PrintJob.cpp line 1230:
>> 
>>> 1228:     jdouble imgY = (jdouble) ((yPixelOrg * 72)/(jdouble) yPixelRes);
>>> 1229:     jdouble imgWid = (jdouble) ((imgPixelWid * 72)/(jdouble) 
>>> xPixelRes);
>>> 1230:     jdouble imgHgt = (jdouble) ((imgPixelHgt * 72)/(jdouble) 
>>> yPixelRes);
>> 
>> Suggestion:
>> 
>>     jdouble imgX = (jdouble)((xPixelOrg * 72)/(jdouble)xPixelRes);
>>     jdouble imgY = (jdouble)((yPixelOrg * 72)/(jdouble)yPixelRes);
>>     jdouble imgWid = (jdouble)((imgPixelWid * 72)/(jdouble)xPixelRes);
>>     jdouble imgHgt = (jdouble)((imgPixelHgt * 72)/(jdouble)yPixelRes);```
>
> I don't think I can commit this as there are 3 backticks at the end there :P

Apologies. The point was that this was formatting changes that were not needed 
and should be reverted.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/15096#discussion_r1537238932

Reply via email to