D17599: Confirm closing muliple tabs.

[Dominik Haumann]

dhaumann added inline comments.

INLINE COMMENTS

> katemainwindow.cpp:1198
> +    // by the session manager because the session is closed
> +    bool closedByUser = true;
> +    if (qApp->isSavingSession()) {

Much better would be a shorter notation where you also can use const:

  const bool closedByUser = !qApp->isSavingSession();

REPOSITORY
  R40 Kate

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

To: shubham, dhaumann, #kate, sars
Cc: ngraham, pino, kwrite-devel, hase, michaelh, demsking, cullmann, sars, dhaumann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwrite-devel/attachments/20190109/6234e03f/attachment.html>