<table><tr><td style="">kossebau 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/D11250">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/D11250#inline-56445">View Inline</a><span style="color: #4b4d51; font-weight: bold;">aacid</span> wrote in <span style="color: #4b4d51; font-weight: bold;">dbusabstractadaptor.h:32</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">This seems like random glue code, wouldn't this make more sense in Qt or in KF5?</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Possibly. Similar code working around the missing Qt feature is found across code. Actually one would expect this indeed with Qt, but the one known related bug report <a href="https://bugreports.qt.io/browse/QTBUG-48008" class="remarkup-link" target="_blank" rel="noreferrer">https://bugreports.qt.io/browse/QTBUG-48008</a> got for whatever reason a harsh wontfix.<br />
Not my  playing ground, so I escaped into manual workaround like found elsewhere. Will be happy to have something in Qt or KF5, but do not have resources for that.</p></div></div><br /><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/D11250#inline-56452">View Inline</a><span style="color: #4b4d51; font-weight: bold;">aacid</span> wrote in <span style="color: #4b4d51; font-weight: bold;">mprismediaplayer2player.cpp:58</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">No, you can not, file can change at any moment.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Also during presentation? Okay. Any change signal you would recommend to catch here? Would encode some thumbnail versioning then to ensure uniquness.</p></div></div><br /><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/D11250#inline-56454">View Inline</a><span style="color: #4b4d51; font-weight: bold;">aacid</span> wrote in <span style="color: #4b4d51; font-weight: bold;">mprismediaplayer2player.cpp:304</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">Why do you need that?</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">To avoid creating repeated requests for the same pixmap.</p>

<p style="padding: 0; margin: 8px;">Challenge I see: I need to get a thumbnail per page. To speed-up things, it will be done only for pages currently visited. But given the request is processed async, when a user switches pages while the requests are still on-going, when entering the same page multiple times multiple requests would be created, one per visit.</p>

<p style="padding: 0; margin: 8px;">Is there some internal mechanism to squash such requests?</p></div></div><br /><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/D11250#inline-56449">View Inline</a><span style="color: #4b4d51; font-weight: bold;">aacid</span> wrote in <span style="color: #4b4d51; font-weight: bold;">mprismediaplayer2player.h:99</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">are this ugly uppercase tyied to the spec? Isn't there like a magic way in Qt to create the api from the xml spec?</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Yes, needed due to using <tt style="background: #ebebeb; font-size: 13px;">sessionBus.registerObject(mediaPlayer2ObjectPath(), this, QDBusConnection::ExportAdaptors);</tt>, and D-Bus methods should be Uppercase and so does MPRIS.<br />
 I only know <tt style="background: #ebebeb; font-size: 13px;">qt5_add_dbus_interface()</tt> for creating nice-Qt-style API for interfaces to be used with remote objects. For local actual implementations of interfaces I am not aware how to avoid that, the other code I saw doing MPRIS implementations did things the same way as here. Might have missed something, but done to my best knowledge.</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/D11250">https://phabricator.kde.org/D11250</a></div></div><br /><div><strong>To: </strong>kossebau, Okular<br /><strong>Cc: </strong>aacid, pino, rkflx, KDE Connect, michaelweghorn, ngraham<br /></div>