On Tue, 13 Sep 2022 19:48:47 GMT, Alexey Ivanov <[email protected]> wrote:
>> Magnus Ihse Bursie has updated the pull request with a new target base due >> to a merge or a rebase. The pull request now contains 46 commits: >> >> - Merge branch 'master' into typos-in-java.desktop >> - Revert changes in libjpeg >> - Revert changes in libfreetype >> - Update src/java.desktop/windows/classes/sun/awt/windows/WPrinterJob.java >> >> Co-authored-by: Alexey Ivanov >> <[email protected]> >> - Update src/java.desktop/unix/classes/sun/awt/X11/XBaseMenuWindow.java >> >> Co-authored-by: Alexey Ivanov >> <[email protected]> >> - Update src/java.desktop/unix/classes/sun/awt/X11/XBaseMenuWindow.java >> >> Co-authored-by: Alexey Ivanov >> <[email protected]> >> - Update src/java.desktop/share/classes/sun/swing/SwingUtilities2.java >> >> Co-authored-by: Alexey Ivanov >> <[email protected]> >> - Update src/java.desktop/share/classes/sun/swing/SwingUtilities2.java >> >> Co-authored-by: Alexey Ivanov >> <[email protected]> >> - Update src/java.desktop/share/classes/sun/awt/image/ImagingLib.java >> >> Co-authored-by: Alexey Ivanov >> <[email protected]> >> - Update src/java.desktop/share/classes/sun/awt/image/ImagingLib.java >> >> Co-authored-by: Alexey Ivanov >> <[email protected]> >> - ... and 36 more: https://git.openjdk.org/jdk/compare/6a1e98cb...1ce28314 > > src/java.desktop/macosx/native/libawt_lwawt/java2d/metal/MTLLayer.m line 87: > >> 85: J2dTraceLn4(J2D_TRACE_VERBOSE, >> 86: "MTLLayer.blitTexture: uninitialized (mtlc=%p, >> javaLayer=%p, buffer=%p, device=%p)", self.ctx, >> 87: self.javaLayer, self.buffer, ctx.device); > > Is this an intended change? Or does it come from a merge? > > May I suggest wrapping the line after the format string before `self.ctx`? It > would make it easier to read. This is from the merge. The only difference wrt to current code is "devide" => "device". I can change wrapping if you request it, but I'd prefer to only fix typos. ------------- PR: https://git.openjdk.org/jdk/pull/8328
