D23667: Add == and != operators to KIO::UDSEntry
Pino Toscano
noreply at phabricator.kde.org
Tue Sep 3 05:38:17 BST 2019
pino added a comment.
Please move the implementations in the cpp file, otherwise it will be impossible to change/fix the implementation later on in a binary compatible way.
Also:
- both the operators ought to be const, since they do not mutate the object (and otherwise they cannot be used to compare const objects)
- please fix the style in the declarations: `const UDSEntry& other` -> `const UDSEntry &other`
REPOSITORY
R241 KIO
REVISION DETAIL
https://phabricator.kde.org/D23667
To: meven, #frameworks
Cc: pino, ngraham, kde-frameworks-devel, LeGast00n, GB_2, michaelh, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190903/586671ce/attachment-0001.html>
More information about the Kde-frameworks-devel
mailing list