D28909: smb: port to Result system to force serialization of error/finish condition
Méven Car
noreply at phabricator.kde.org
Sun Apr 19 07:13:24 BST 2020
meven added inline comments.
INLINE COMMENTS
> kio_smb_browse.cpp:184
> } else {
> - error(passwordError, url.toString());
> }
Empty else clause leftover
> kio_smb_file.cpp:169
> if (errNum == EACCES)
> - error(KIO::ERR_ACCESS_DENIED, m_openUrl.toDisplayString());
> - else
> - error(KIO::ERR_DOES_NOT_EXIST, m_openUrl.toDisplayString());
> - return;
> + return Result::fail(KIO::ERR_ACCESS_DENIED, m_openUrl.toDisplayString());
> + return Result::fail(KIO::ERR_DOES_NOT_EXIST, m_openUrl.toDisplayString());
add {}
REPOSITORY
R320 KIO Extras
REVISION DETAIL
https://phabricator.kde.org/D28909
To: sitter, dfaure
Cc: meven, kde-frameworks-devel, kfm-devel, azyx, nikolaik, pberestov, iasensio, fprice, LeGast00n, cblack, fbampaloukas, alexde, Codezela, feverfew, michaelh, spoorun, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp, rdieter, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20200419/2fd2aecf/attachment.htm>
More information about the kfm-devel
mailing list