kfunk added a comment.
In https://phabricator.kde.org/D5865#112766, @rjvbb wrote: > KDE is FOSS not bound to Microsoft by any corporate buy-in or whatever, right? What non-sense is this? Please stay on topic. There's a benefit we make sure KDE software is compiling under MSVC given its popularity on Windows. > If there's a bug to report it's the lack of standard compliance in MSVC - how have MS reacted to such reports? ... and you're going to have them fix older MSVC versions to support your use-case? This discussion is such a waste of time really. Please face the reality and show some pragmatism: Named operators do not work on MSVC prior to MSVC 2017, period. +1 on Thiago's solution, for the good. Let's just print a warning for MSVC prior to version 2017: > That's a fine solution. If you're going to print a warning and you're serious about cross-platformness support, I suggest printing the warning for ALL compilers: "Warning: using C++ operator names is not compatible with MSVC prior to version 2017 (19.1). You should reconsider using this macro if your code is meant to be cross-platform". REPOSITORY R240 Extra CMake Modules REVISION DETAIL https://phabricator.kde.org/D5865 To: rjvbb, #frameworks, #build_system, cgilles, kfunk Cc: thiago, kfunk
