RFR: 8260931: Implement JEP 382: New macOS Rendering Pipeline [v3]
Gerard Ziemski
gziemski at openjdk.java.net
Mon Feb 8 17:26:43 UTC 2021
On Mon, 8 Feb 2021 17:15:25 GMT, Gerard Ziemski <gziemski at openjdk.org> wrote:
>> The file in `RenderPerfTest` should have a GPLv2 license header (no Classpath). I filed [JDK-8261273](https://bugs.openjdk.java.net/browse/JDK-8261273) and also highlighted a couple examples below.
>
> General comment - I am not sure I like the `MTL` prefix acronym in names (ex. `sun.java2d.metal.MTLVolatileSurfaceManager`).
>
> I think you tried to match the `CGL`, but that is a real acronym that stands for "Core Graphics Layer" (I think).
>
> `MTL` on the other hand is no acronym. I can see `ML` for "Metal Layer" I suppose, but also just `Metal` would work just fine.
I'm in the process of reviewing this feature, but there is lots of code to go through - please wait for my review.
-------------
PR: https://git.openjdk.java.net/jdk/pull/2403
More information about the build-dev
mailing list