D22323: Code cleanups

Harald Sitter noreply at phabricator.kde.org
Mon Jul 8 11:25:47 BST 2019


sitter accepted this revision.
sitter added a comment.
This revision is now accepted and ready to land.


  I do wonder if it wouldn't make sense to make the two helpers into a new class DrKonqiLauncher or some such and then delegate this entire decision to the class. The way this was before the diff is weird, and the way it is after the diff is still weird IMHO. Also the two objects that are allocated in the two methods aren't properly deleted anywhere it seems.

INLINE COMMENTS

> main.cpp:56
> +namespace {
> +    void openDrKonqiDialog () {
> +        DrKonqiDialog *w = new DrKonqiDialog();

excess space before `(`

REPOSITORY
  R871 DrKonqi

BRANCH
  codeCleanups

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

To: tcanabrava, sitter
Cc: sitter, plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190708/fb2040f3/attachment.html>


More information about the Plasma-devel mailing list