RFR: 8329340: Remove unused libawt code [v2]

Alexey Ivanov aivanov at openjdk.org
Tue Apr 2 17:30:13 UTC 2024


On Tue, 2 Apr 2024 16:55:27 GMT, Daniel Jeliński <djelinski at 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


More information about the client-libs-dev mailing list