RFR: 8307246 : Printing: banded raster path doesn't account for device offset values [v6]
Phil Race
prr at openjdk.org
Fri Jan 26 20:35:37 UTC 2024
On Thu, 25 Jan 2024 11:01:48 GMT, vtstydev <duke at openjdk.org> wrote:
>> More correct way to take in consideration nonzero PHYSICALOFFSETX, PHYSICALOFFSETY of device for banded-raster printing loop. Only on Windows platform under certain conditions real device prints shifted image on paper.
>
> vtstydev has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains one new commit since the last revision:
>
> Add test for comparing offsets of images
Very close (!) I see just a couple of minor things I need you to fix.
test/jdk/java/awt/print/PrinterJob/ImagePrinting/AlphaPrintingOffsets.java line 2:
> 1: /*
> 2: * Copyright (c) 2007, 2022, Oracle and/or its affiliates. All rights reserved.
I guess you copied the header including "2007, 2022," from some other file.
This is a new test so it should just be "2024,"
test/jdk/java/awt/print/PrinterJob/ImagePrinting/AlphaPrintingOffsets.java line 28:
> 26: import javax.print.attribute.standard.Sides;
> 27: import javax.swing.*;
> 28: import java.awt.*;
We don't use wildcard imports - the example I pasted had none !
So please explicitly list all imports and we "sort" these. generally alphabetically at least for the same module.
So the order of the packages should be
java.awt
java.awt.event
java.awt.print
javax.print
javax.swing
-------------
PR Review: https://git.openjdk.org/jdk/pull/17030#pullrequestreview-1846502348
PR Review Comment: https://git.openjdk.org/jdk/pull/17030#discussion_r1468117611
PR Review Comment: https://git.openjdk.org/jdk/pull/17030#discussion_r1468119702
More information about the client-libs-dev
mailing list