On Tue, 2 Apr 2024 16:55:27 GMT, Daniel Jeliński <djelin...@openjdk.org> wrote:
>> src/java.desktop/windows/native/libawt/windows/awt_Debug.cpp line 111:
>>
>>> 109: * DTRACE print callback to dump window's update region bounding
>>> rectangle
>>> 110: */
>>> 111: void DumpUpdateRectangle(const char * file, int line, int argc, const
>>> char * fmt, va_list arglist) {
>>
>> Can these functions be useful for debugging?
>>
>> Does it make sense to hide them unless `DEBUG` is defined?
>
> they were `if defined(DEBUG)` already; I can keep them if you think it makes
> sense.
I don't have any strong preference. These may be useful in rare cases where
painting doesn't work as expected…
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/18553#discussion_r1548287746
