Review Request 115378: Resolve some FIXMEs related to MSVC and Intel

Alex Merry kde at randomguy3.me.uk
Tue Feb 4 12:26:11 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/115378/
-----------------------------------------------------------

(Updated Feb. 4, 2014, 12:26 p.m.)


Review request for Build System, Extra Cmake Modules, KDE Frameworks, and Andrius da Costa Ribas.


Changes
-------

Another FIXME (partially) cleared up.


Repository: extra-cmake-modules


Description
-------

Use the MSVC variable directly, instead of the compiler ID

This is cleaner and easier to read.

Resolve some FIXMEs related to MSVC and Intel

In particular, MSVC (and Intel on Windows) have no equivalent of the
-std flag to set the language standard, and Intel does not appear to
produce the warnings that were disabled for MSVC.


Diffs (updated)
-----

  kde-modules/KDECompilerSettings.cmake 335e1270d19f8342e41b22e7081dea3f7ac0fbfc 

Diff: https://git.reviewboard.kde.org/r/115378/diff/


Testing
-------


Thanks,

Alex Merry

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140204/a48540d7/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list