<table><tr><td style="">romangg added inline comments.
</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/D27626">View Revision</a></tr></table><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/D27626#inline-156185">View Inline</a><span style="color: #4b4d51; font-weight: bold;">dvratil</span> wrote in <span style="color: #4b4d51; font-weight: bold;">output_model.h:90</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">Because in <tt style="background: #ebebeb; font-size: 13px;">output_model.cpp:224</tt> we do <tt style="background: #ebebeb; font-size: 13px;">m_outputs.insert(i, Output(output, pos))</tt>, which can make use of move semantics.</p>

<p style="padding: 0; margin: 8px;">Making move operations <tt style="background: #ebebeb; font-size: 13px;">noexcept</tt> is part of <a href="https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#c66-make-move-operations-noexcept" class="remarkup-link" target="_blank" rel="noreferrer">Cpp Core Guidelines</a> and because <tt style="background: #ebebeb; font-size: 13px;">QSharedPointer</tt>'s move ctor is <tt style="background: #ebebeb; font-size: 13px;">noexcept</tt> and <tt style="background: #ebebeb; font-size: 13px;">QPoint</tt> is trivially movable, the move operations here are also <tt style="background: #ebebeb; font-size: 13px;">noexcept</tt>.</p>

<p style="padding: 0; margin: 8px;">If it has any benefit here, since we possibly don't even have exceptions enabled in kscreen, is a question - I wrote it by force of habit.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Thanks for the interesting explanation. I find it impractical that the usefulness of the noexcept keyword here depends on internals of the Qt classes. One would wish this is advertised to the consumer without knowing the internals. Or is it somewhere I'm not aware of?</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R104 KScreen</div></div></div><br /><div><strong>BRANCH</strong><div><div>master</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D27626">https://phabricator.kde.org/D27626</a></div></div><br /><div><strong>To: </strong>dvratil, Plasma, romangg<br /><strong>Cc: </strong>romangg, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>