D13826: Support activities when opening files

Simone Scalabrino noreply at phabricator.kde.org
Mon Jul 9 10:13:35 BST 2018

sscalabrino added a comment.

  In D13826#289245 <https://phabricator.kde.org/D13826#289245>, @cullmann wrote:
  > Think that is ok, if the given comments are implemented.
  > An alternative approach for the API would btw. to have just a function that tells you "I am on activity X", then one doesn't need to send stringlists around and check for contains.
  Thanks for the comment. The problem is that it is possible that the window is on more than a single activity (e.g., there are three activities, A, B and C, and a single Kate instance is in both A and C at the same time), that's why I used a StringList. Indeed, I implemented that as you suggested at the beginning, but then I noticed this problem. Also, the check is done only when the user opens a new file, and the list of activities should not be that long. In general, I think that the overhead is acceptable, and it allows us to have a complete support for activities.
  Anyways, I will implement all your comments ASAP and I will update the patch.
  Thanks for the review.

  R40 Kate


To: sscalabrino, #kate, #ktexteditor, cullmann, dhaumann
Cc: ngraham, kwrite-devel, michaelh, kevinapavew, demsking, cullmann, sars, dhaumann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwrite-devel/attachments/20180709/ddaa59d4/attachment-0001.html>

More information about the KWrite-Devel mailing list