<div dir="ltr"><div class="gmail_default" style="font-family:monospace,monospace;font-size:small"><br></div><div class="gmail_extra"><br><div class="gmail_quote">On 14 November 2016 at 10:45, Martin Gräßlin <span dir="ltr"><<a href="mailto:mgraesslin@kde.org" target="_blank">mgraesslin@kde.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi framework devs,<br>
<br>
recently we started to see the first patches for frameworks to silence warnings for not used features of C++11. In particular to add override to methods of inheriting classes.<br>
<br>
Now I find this weird from the perspective of our C++ requirements. On the one side we say that we are not allowed to use these features to still support non-C++11 compilers on the other side we see that compilers already generate warnings if you don't use the C++11 features.<br>
<br>
In that particular case I do not think that <div style="font-family:monospace,monospace;font-size:small;display:inline" class="gmail_default">​​</div>Q_DECL_OVERRIDE is a solution. It's an ugly hack and for the frameworks I maintain I gave a -2 on the review. I think this needs a general solution of either not adding or adding override, but not the Qt hack.<br>
<br>
I think this is a sign that we need to move on. We cannot continue with the state we are in. It's too much a hassle for developers:<br>
* false-positive compile warnings<br>
* no way to check which features are allowed or not<br>
* no warnings if a not allowed feature is used<br>
* no CI system in place to ensure our rules<br>
<br>
Given that I want to suggest that we remove all compiler restrictions and allow the full feature set of C++11. If someone still thinks we need to support the compilers not supporting C++11, I would like to see a plan on how we can improve the developer story, especially how to address the last two points in my list above.<br>
<br>
Opinions?<br></blockquote><div><br><div style="font-family:monospace,monospace;font-size:small;display:inline" class="gmail_default">​+1<br>I think it's time to move on for non-KF5 repos​</div> <div style="font-family:monospace,monospace;font-size:small;display:inline" class="gmail_default">​as well just to be consistent.<br></div><div style="font-family:monospace,monospace;font-size:small;display:inline" class="gmail_default">​Mayb a hint on kde-devel or a blog entry could help with that. ​<br><br></div><div style="font-family:monospace,monospace;font-size:small;display:inline" class="gmail_default">PS: TODO: update the requirements at <a href="https://community.kde.org/Frameworks/Policies#Frameworks_compiler_requirements_and_C.2B.2B11">https://community.kde.org/Frameworks/Policies#Frameworks_compiler_requirements_and_C.2B.2B11</a> and remove the suggestion to use ​Q_DECL_OVERRIDE.<br></div></div></div><br clear="all"><br>-- <br><div class="gmail_signature">regards, Jaroslaw Staniek<br><br>KDE:<br>: A world-wide network of software engineers, artists, writers, translators<br>: and facilitators committed to Free Software development - <a href="http://kde.org" target="_blank">http://kde.org</a><br>Calligra Suite:<br>: A graphic art and office suite - <a href="http://calligra.org" target="_blank">http://calligra.org</a><br>Kexi:<br>: A visual database apps builder - <a href="http://calligra.org/kexi" target="_blank">http://calligra.org/kexi</a><br>Qt Certified Specialist:<br>: <a href="http://www.linkedin.com/in/jstaniek" target="_blank">http://www.linkedin.com/in/jstaniek</a></div>
</div></div>