<table><tr><td style="">vkrause 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/D15467">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/D15467#inline-83321">View Inline</a><span style="color: #4b4d51; font-weight: bold;">nicolasfella</span> wrote in <span style="color: #4b4d51; font-weight: bold;">semanticurlhandler.cpp:500</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">openInApp doesn't close either. <a href="https://phabricator.kde.org/p/vkrause/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@vkrause</a> is this intended?</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">The explicit close looks correct to me, f auto-closes (and thus flushes to disk) when leaving the scope, but technically we use it before that already. Very unlikely, but in theory it seems possible that the other app gets scheduled in before we flushed the content.</p>

<p style="padding: 0; margin: 8px;">Would need to be fixed in the other place too then.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R81 KDE PIM Addons</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D15467">https://phabricator.kde.org/D15467</a></div></div><br /><div><strong>To: </strong>nicolasfella, vkrause, mlaurent<br /><strong>Cc: </strong>mlaurent, kde-pim, dvasin, rodsevich, winterz, vkrause, knauss, dvratil<br /></div>