<table><tr><td style="">rkflx 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/D8838" rel="noreferrer">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Another option is just pushing the fix for the second problem and not for the first one</p></blockquote>
<p>After testing again, "patch w/o timer" now gives "M-D-T", i.e. the last M is not there anymore. I suspected I could find a window size or map area to break it again (as noted before), and indeed, I then got M-D-M-T, which is different to the one from above again…</p>
<p>For now I'd say your suggestion to only fix the second problem is probably the best we can do for the moment, it improves the situation in some cases at least.</p>
<p>The cancelling, resize events uncertainty, blank rotated pages and the seldom extra "M"s are most likely best handled in other patches.</p></div></div><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/D8838#inline-40491" rel="noreferrer">View Inline</a><span style="color: #4b4d51; font-weight: bold;">generator.cpp:718</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; "> <span class="p">.</span><span class="n">arg</span><span class="p">(</span> <span class="n">req</span><span class="p">.</span><span class="n">pageNumber</span><span class="p">()</span> <span class="p">);</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(251, 175, 175, .7);"> <span class="n">str</span> <span style="color: #aa2211"><<</span> <span class="n">qPrintable</span><span class="p">(</span> <span class="n">s</span> <span class="p">);</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);"> <span class="n">str</span> <span style="color: #aa2211"><<</span> <span class="n">qPrintable</span><span class="p">(</span> <span class="n">s</span> <span class="p">)<span class="bright"></span></span><span class="bright"> </span><span style="color: #aa2211"><span class="bright"><<</span></span><span class="bright"> </span><span class="n"><span class="bright">req</span></span><span class="bright"></span><span class="p"><span class="bright">.</span></span><span class="bright"></span><span class="n"><span class="bright">normalizedRect</span></span><span class="bright"></span><span class="p"><span class="bright">()</span>;</span>
</div><div style="padding: 0 8px; margin: 0 4px; "> <span style="color: #aa4000">return</span> <span class="n">str</span><span class="p">;</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Interesting, how would I get this debug output? (modifying Okular's entries in <tt style="background: #ebebeb; font-size: 13px;">kdebugsettings</tt> did not print this line).</p>
<p style="padding: 0; margin: 8px;">Still, this looks a bit like stray debug output, or should this get its own <tt style="background: #ebebeb; font-size: 13px;">.arg</tt> explanation?</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/D8838" rel="noreferrer">https://phabricator.kde.org/D8838</a></div></div><br /><div><strong>To: </strong>aacid<br /><strong>Cc: </strong>rkflx, Okular, gassaf, ngraham, aacid<br /></div>