On Mon, 16 Dec 2024 21:51:35 GMT, Jeremy <[email protected]> wrote:

>> '500 iterations' looks high, maybe 100?
>> It is a recursion ? You could try max recursion depth + 3 samples (first, 
>> middle, last) ?
>> 
>> Or this magic threshold could be tunable ac runtime using special static 
>> getter using a System property + default value, used by constant declaration 
>> (final static is mandatory):
>> 
>> https://github.com/bourgesl/jdk-official/blob/d0b770c938be6b5b3a0176148265ef33184a9b8b/src/java.desktop/share/classes/sun/java2d/marlin/MarlinProperties.java#L50
>> 
>> https://github.com/bourgesl/jdk-official/blob/d0b770c938be6b5b3a0176148265ef33184a9b8b/src/java.desktop/share/classes/sun/java2d/marlin/MarlinProperties.java#L303
>
> @bourgesl before I try to integrate these changes: does my response above 
> make sense/satisfy your concern?

LGTM

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21962#discussion_r1887982779

Reply via email to