<table><tr><td style="">aheinecke 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/D10048" rel="noreferrer">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/D10048#inline-50758" rel="noreferrer">View Inline</a><span style="color: #4b4d51; font-weight: bold;">aacid</span> wrote in <span style="color: #4b4d51; font-weight: bold;">document.cpp:1139</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">This is problematic since it will leave a dangling event pointer for any Action::Script action executed through Document::processAction that doesn't come from this function.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Sorry I can't follow you here.</p>

<p style="padding: 0; margin: 8px;">If an action is processed from somewhere else the scripter won't have an event set and the event pointer in the scripter is null.</p>

<p style="padding: 0; margin: 8px;">If the processAction is triggered here for text fields (and probably more in the future) we "reset" the event pointer after it was set in the scripter in line 1148 after the action is processed and keep using the event here until it goes out of scope and is destroyed by the shared pointer.</p>

<p style="padding: 0; margin: 8px;">I don't see a codepath were we leave it dangling.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R223 Okular</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D10048" rel="noreferrer">https://phabricator.kde.org/D10048</a></div></div><br /><div><strong>To: </strong>aheinecke, Okular<br /><strong>Cc: </strong>aacid, michaelweghorn, ngraham<br /></div>