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