RFR: JDK-8323695 RenderPerf (2D) enhancements (23.12)

[Phil Race]

On Fri, 12 Jan 2024 21:34:32 GMT, Laurent Bourgès <lbourges at openjdk.org> wrote:

> - new executor modes (buffer & volatile)
> - support parallel rendering on several frames and / or screens
> - added robot calibration + optionally use rounded duration to min frame latency (120Hz)
> - give more statistics in verbose (-v)
> - added new command-line arguments
> - added version + help information

test/jdk/performance/client/RenderPerfTest/src/renderperf/RenderPerfTest.java line 204:

> 202:                     enabled ? RenderingHints.VALUE_TEXT_ANTIALIAS_LCD_HRGB
> 203:                             : RenderingHints.VALUE_TEXT_ANTIALIAS_DEFAULT);
> 204:         }

Although it makes no practical difference, I am wondering why in this case, enabled==false sets "DEFAULT" and in the case above it sets "OFF" ?

test/jdk/performance/client/RenderPerfTest/src/renderperf/RenderPerfTest.java line 460:

> 458:             if (id % 100 != 0) return;
> 459:             if (font == null) {
> 460:                 font = new Font("LucidaGrande", Font.PLAIN, 32);

I see this is pre-existing, but we should change this.
That's a font available only on macOS. You should either look up a font to use on the system at run time,
or use a logical font.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17408#discussion_r1454084838
PR Review Comment: https://git.openjdk.org/jdk/pull/17408#discussion_r1454088478