<table><tr><td style="">tobiasdeiminger accepted this revision.<br />tobiasdeiminger added a comment.<br />This revision is now accepted and ready to land.
</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/D14752">View Revision</a></tr></table><br /><div><div><p>Here's a <a href="https://bugs.kde.org/attachment.cgi?id=71098" class="remarkup-link" target="_blank" rel="noreferrer">PDF</a> and <a href="https://bugs.kde.org/attachment.cgi?id=71099" class="remarkup-link" target="_blank" rel="noreferrer">LaTex source</a>, suitable to trigger and test OkularLinkType::setAnnotation in resolveMediaLinks.</p>

<p>Everything still works fine if patch is applied:</p>

<ol class="remarkup-list">
<li class="remarkup-list-item">Video starts automatically in presentation mode, before and after highlighting + ctrl-s</li>
<li class="remarkup-list-item">Watching this procedure with valgrind shows no memory leaks</li>
</ol>

<p>So here's an approve from my side, thanks Albert for the patch.</p>

<p>(video doesn't autostart in page view mode, no matter if patch is applied or not - should it?)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R223 Okular</div></div></div><br /><div><strong>BRANCH</strong><div><div>arcpatch-D14752</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D14752">https://phabricator.kde.org/D14752</a></div></div><br /><div><strong>To: </strong>aacid, tobiasdeiminger<br /><strong>Cc: </strong>tobiasdeiminger, sander, okular-devel, ngraham, aacid<br /></div>