D18939: KMenuEdit: add search/filter bar
Aidan Sojourner
noreply at phabricator.kde.org
Wed Feb 13 01:08:18 GMT 2019
tuxxi added inline comments.
INLINE COMMENTS
> ngraham wrote in kmenuedit.cpp:143
> If there's nothing specifically in the style guidelines about it, it's best to follow the existing coding style. Nothing else here uses `auto` with `new` constructors, so we should follow the same style for new code.
>
> KDE software is multi-generational and it's important that each individual developer not use their own personal preferred style instead of following the existing style because that leads to the whole codebase becoming an inconsistent mess over time. Cleanup can be desirable, but that should happen separately, in its own patch, so it can be discussed on its own merits.
Okay, makes sense. I'll try to be more atomic :)
REPOSITORY
R103 KMenu Editor
REVISION DETAIL
https://phabricator.kde.org/D18939
To: tuxxi, ngraham, #plasma, cfeck
Cc: ngraham, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190213/f832300d/attachment.html>
More information about the Plasma-devel
mailing list