D7011: Extract lineedit password

Friedrich W. H. Kossebau noreply at phabricator.kde.org
Mon Jul 31 16:24:20 UTC 2017


kossebau added inline comments.

INLINE COMMENTS

> passwordlineedit.h:27
> +class PasswordLineEditPrivate;
> +class KWIDGETSADDONS_EXPORT PasswordLineEdit : public QWidget
> +{

Surprised to see no K-prefix being used here? Why that?

Please add one and rename to `KPasswordLineEdit`:

- for consistency with the existing classes from kwidgetaddons/kde frameworks
- to protect against potential clashes, even if rare chance, with existing code linking to kwidgetaddons, which assumes using unprefixed/namespace-less names is safe in own code and might use that name for something else already.
- namespace-less class name lack any hint to origin, but rather used for project-internal code, so confuses code readers

REVISION DETAIL
  https://phabricator.kde.org/D7011

To: mlaurent, cfeck, dfaure, elvisangelaccio
Cc: kossebau, elvisangelaccio, #frameworks
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20170731/a8e49711/attachment.html>


More information about the Kde-frameworks-devel mailing list