On Thu, 23 Jan 2025 20:31:03 GMT, Phil Race <p...@openjdk.org> wrote:

> The compiler must have been confused.

Definitely, but I don't blame it. Look at this construct:

#define IMGEXTERN
#include "imageInitIDs.h"

There is really no way for the poor compiler to understand that in this case, 
it was really intentional to define variables in a .h file. And once again, 
this is a clever construct. If we had used it in all our code base I would not 
have complained. But when we do it in just a single place, it breaks 
expectations, and it is better to align the code to how we do everywhere else.

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

PR Comment: https://git.openjdk.org/jdk/pull/23236#issuecomment-2611199045

Reply via email to