<table><tr><td style="">mpyne added a comment.
</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/D5966" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>The changeset is fine to go in (once the duplicated semicolon is fixed).</p>

<p>As for C++11, I'm not aware that Qt requires C++11 as a blanket policy.  Instead they require specific compilers, and then we can use the C++11 features supported by that compiler.</p>

<p>According to the <a href="http://doc.qt.io/qt-5/supported-platforms-and-configurations.html" class="remarkup-link" target="_blank" rel="noreferrer">Qt documentation</a>, Qt 5.8 is the first Qt release to require at least GCC 4.8 across the board, on configurations using GCC.  I believe that was the first GCC where <tt style="background: #ebebeb; font-size: 13px;">thread_local</tt> is fully supported.  Note that we do try to support MS Windows though, and MSVC as distributed with VS 2013 is the minimum requirement for both Qt 5.8 and current Qt development.  But MSVC doesn't fully support <tt style="background: #ebebeb; font-size: 13px;">thread_local</tt> until <a href="https://msdn.microsoft.com/en-us/library/hh567368.aspx#concurrencytable" class="remarkup-link" target="_blank" rel="noreferrer">VS 2015</a>.  So it will probably be at least another couple of years before we can rely on that feature to be available.</p></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D5966#inline-24675" rel="noreferrer">View Inline</a><span style="color: #4b4d51; font-weight: bold;">krandom.cpp:46</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; ">        <span style="color: #aa4000">unsigned</span> <span style="color: #aa4000">int</span> <span class="n">seed</span><span class="p">;</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(251, 175, 175, .7);">        <span class="n">init<span class="bright"></span></span><span class="bright"> </span><span style="color: #aa2211"><span class="bright">=</span></span><span class="bright"> </span><span style="color: #304a96">true</span><span class="p">;</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">        <span class="n">init<span class="bright">ialized_threads</span></span><span class="bright"></span><span class="p"><span class="bright">.</span></span><span class="bright"></span><span class="n"><span class="bright">setLocalData</span></span><span class="bright"></span><span class="p"><span class="bright">(</span></span><span style="color: #304a96">true<span class="bright"></span></span><span class="bright"></span><span class="p"><span class="bright">);</span>;</span>
</div><div style="padding: 0 8px; margin: 0 4px; ">        <span class="n">QFile</span> <span style="color: #004012">urandom</span><span class="p">(</span><span class="n">QStringLiteral</span><span class="p">(</span><span style="color: #766510">"/dev/urandom"</span><span class="p">));</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">There are two semicolons on this line, only one is needed. :)</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R244 KCoreAddons</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D5966" rel="noreferrer">https://phabricator.kde.org/D5966</a></div></div><br /><div><strong>To: </strong>tfry, dfaure<br /><strong>Cc: </strong>mpyne, tfry, rjvbb, Frameworks<br /></div>