Closed IMAN4K closed 4 months ago
I'm quite okay with the keyword changes. However, I'm wondering about the add_definitions()
. I feel like this is something that the consuming target should define if necessary.
In any case, that should be done via target_compile_definitions()
instead.
@Tectu Yeah it could be an option
In the case of using 3rd parties like KDBindings to remove name clashes simply
QT_NO_KEYWORDS
must be applied.Best Regards, Iman.