D5865: Add missing KDE_ENABLE_NAMED_OPERATORS function
Kevin Funk
noreply at phabricator.kde.org
Tue May 30 09:52:08 UTC 2017
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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20170530/7b7bd57a/attachment.html>
More information about the Kde-buildsystem
mailing list