<table><tr><td style="">rjvbb edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-bfkjqwav5anvz3w/" rel="noreferrer">(Show Details)</a><br />rjvbb edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-diergaygsgolvyt/" rel="noreferrer">(Show Details)</a><br />rjvbb set the repository for this revision to R240 Extra CMake Modules.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D5865" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><div style="padding: 8px 0;">...</div>This patch adds a function to (re)enable them in a cross-platform fashion in projects where they are required, the same way C++ exceptions are handled.<br />
<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">I have implemented the MSVC-specific part to the best of my knowledge but cannot test it and have not yet been able to figure out what the exact MSVC_VERSION value is to test against</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">The MSVC-specific part will (hopefully) activate named operator support in MSVC 2017 and later, without side-effects (again, hopefully)</span>. <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">"GREATER 1900" should be correct but may not be specific enough</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">For earlier versions a warning is printed as well as when using other compilers</span>.</div></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;">Tested on Mac and Linux (with digiKam 5.5.0).<br />
<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Someone should test the value of MSVC_VERSION with MSVC 2015 Update 3 - in case I cannot get that information via the CMake ML</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">The same warning is printed when using an unsupported MSVC version or a compiler supporting `-fnamed-operators`, following Thiago's wording. It might be better to warn against the purpose instead of against using the macro, esp. when an unsupported MSVC is being used</span>.</div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R240 Extra CMake Modules</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D5865" rel="noreferrer">https://phabricator.kde.org/D5865</a></div></div><br /><div><strong>To: </strong>rjvbb, Frameworks, Build System, cgilles, kfunk<br /><strong>Cc: </strong>thiago, kfunk<br /></div>