<table><tr><td style="">rkflx created this revision.<br />rkflx added a reviewer: Frameworks.<br />Restricted Application added a project: Frameworks.<br />rkflx requested review of this revision.
</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/D9987" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Calls to <tt style="background: #ebebeb; font-size: 13px;">QFile::exists()</tt> while starting a Plasma session<br />
resulted in <tt style="background: #ebebeb; font-size: 13px;">Empty filename passed to function</tt> being logged<br />
since Qt 5.10 (0dee566e98f5 in qtbase).</p>

<p>This can be avoided by skipping the check for empty filenames, which<br />
would have returned <tt style="background: #ebebeb; font-size: 13px;">false</tt> in any case. Similarly to what was done in<br />
<a href="https://phabricator.kde.org/R241:549fc81e8e8a47afc5621d1383de5d440fc35f76" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">R241:549fc81e8e8a</a>, also port to the faster <tt style="background: #ebebeb; font-size: 13px;">QFileInfo::exists()</tt>.</p>

<p>Note this does not simply hide a deeper issue, i.e. the actual cause<br />
resulting in the empty variable, because <tt style="background: #ebebeb; font-size: 13px;">libpath.isEmpty()</tt> will be<br />
handled explicitly later on. In the case under investigation<br />
<tt style="background: #ebebeb; font-size: 13px;">/usr/lib64/libexec/org_kde_powerdevil</tt> did not match<br />
<tt style="background: #ebebeb; font-size: 13px;">/usr/lib64/libexec/kf5</tt>, but even if it was in that subfolder there<br />
would have been no corresponding <tt style="background: #ebebeb; font-size: 13px;">libkdeinit5_</tt> anyway.</p>

<p>CCBUG: 388611</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Grep session log for "Empty filename passed to function" after<br />
restarting the Plasma session and compare before/after.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R303 KInit</div></div></div><br /><div><strong>BRANCH</strong><div><div>master</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D9987" rel="noreferrer">https://phabricator.kde.org/D9987</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/kdeinit/kinit.cpp</div></div></div><br /><div><strong>To: </strong>rkflx, Frameworks<br /></div>