<table><tr><td style="">mgallien added subscribers: vhanda, mgallien.<br />mgallien accepted this revision.<br />mgallien 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/D8007" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Sorry for me being late to review your work. I had not noticed it.<br />
Thanks for your work.<br />
I am all for this change.<br />
The code you are removing is not currently covered by the automatic tests and is different from the same code in Okular (generator_pdf.cpp:705).<br />
Even if the result of the current automatic test should not be affected by this change, could you please accept the offer from <a href="https://phabricator.kde.org/p/aacid/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@aacid</a> and test with some of his documents (or better with all) ?</p>

<p>I have another question also related to some bugfixes I have done in KFileMetaData. Do you have an idea how to trigger an update of the Baloo database since your changes may modify the title of documents people have in the Baloo database ?</p>

<p><a href="https://phabricator.kde.org/p/vhanda/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@vhanda</a> do you know if there is something to do to update Baloo database when metadata returned by KFileMetaData are changed even if the file itself did not change ?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R286 KFileMetaData</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D8007" rel="noreferrer">https://phabricator.kde.org/D8007</a></div></div><br /><div><strong>To: </strong>flameeyes, Frameworks, mgallien<br /><strong>Cc: </strong>mgallien, vhanda, ngraham, Frameworks<br /></div>