On 9/23/2019 6:05 PM, Alexander Matveev wrote:
Hi Alexey,
http://cr.openjdk.java.net/~asemenyuk/8231279/webrev.01/src/jdk.jpackage/linux/classes/jdk/jpackage/internal/LinuxDebBundler.java.frames.html
Maybe add new line between 41-42 and remove 43 to align import static
with other static
Hi Alexey,
http://cr.openjdk.java.net/~asemenyuk/8231279/webrev.01/src/jdk.jpackage/linux/classes/jdk/jpackage/internal/LinuxDebBundler.java.frames.html
Maybe add new line between 41-42 and remove 43 to align import static
with other static imports.
Looks fine.
Thanks,
Alexander
On
looks good.
/Andy
On 9/23/2019 7:49 AM, Alexey Semenyuk wrote:
Please review the jpackage fix for bug [1] at [2].
This is a fix for the JDK-8200758-branch branch of the open sandbox
repository (jpackage).
Baseline for the fix is [3] patch.
[1]
Please review the jpackage fix for bug [1] at [2].
This is a fix for the JDK-8200758-branch branch of the open sandbox
repository (jpackage).
Baseline for the fix is [3] patch.
[1] https://bugs.openjdk.java.net/browse/JDK-8231279
[2] http://cr.openjdk.java.net/~asemenyuk/8231279/webrev.01/