<table><tr><td style="">rjvbb updated this revision to Diff 14915.<br />rjvbb 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/D5990" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>The kerfuffle changes (lowlevel) were moved to their own revision, making this a patch that affects the user experience directly.</p>

<p>The #ifdefs were all removed, since the additional code should not have any effect elsewhere but on Mac (QFileOpenEvent might in the future be supported elsewhere too, btw).</p>

<p>The event handler class instance is declared volatile to decrease the chance that the linker will optimise the assignment away because the variable isn't used. Maybe wouldn't happen anyway when using <tt style="background: #ebebeb; font-size: 13px;">new</tt> to initialise a local variable?</p></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D5990?vs=14891&id=14915" rel="noreferrer">https://phabricator.kde.org/D5990?vs=14891&id=14915</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D5990" rel="noreferrer">https://phabricator.kde.org/D5990</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>app/CMakeLists.txt<br />
app/MacOSXBundleInfo.plist.in<br />
app/main.cpp</div></div></div><br /><div><strong>To: </strong>rjvbb, Ark, KDE Applications<br /><strong>Cc: </strong>elvisangelaccio, kde-utils-devel, kde-mac, tctara<br /></div>