On Sunday 06 August 2017 03:30:35 vijay krishnavanshi wrote: > Hi, > > > > 1) kf5 branch contains some commits which have open issues on > > > reviewboard, namely: > > > > > > https://git.reviewboard.kde.org/r/124305/ > > > https://git.reviewboard.kde.org/r/124181/ > > > https://git.reviewboard.kde.org/r/124349/ > > I will look into these changes and fix whatever necessary > > > > 2) kf5 branch contains modification of source code of external > > > bundled libiris library in kopete/protocols/jabber/libiris/. See > > > also file: protocols/jabber/libiris/README_BEFORE_COMMITTING > > > > > > $ git diff origin/master..origin/kf5 -- protocols/jabber/libiris/ > > > > > > Due to synchronization with upstream, we either need to drop > > > these changes or send them to upstream. > > These changes can be dropped and most of the changes are of using > nullptr instead of 0. I will look into this more and give an update > to what will be affected if it is dropped. > > > And 3) > > > > I'm not sure if commit cb97ce4e61baa44d9f401b8853982355ef6e684f > > (Gentle quitting Kopete application) is correct. > > > > It just looks like a workaround some bug in system try area. > > This commit looks okay to me as nothing much is changed in terms of > behavior. In this commit an attempt to close all the windows is made > when we are able to close all window other wise return from the > function. In normal conditions returning in the middle of for loop > does not cause any problem but I will try to verify the same for the > application. I have linked it below to make sure if this is the same > commit we are talking about. > Link : > https://cgit.kde.org/kopete.git/commit/?h=kf5&id=cb97ce4e61baa44d9f40 > 1b8853982355ef6e684f
Previous behaviour is to stay kopete running in systray even last windows is closed. And above commit change behavior from setQuitOnLastWindowClosed(false) to setQuitOnLastWindowClosed(true) which means that Kopete is no longer running in systray after closing last window. That second part of changing break to return is probably OK. -- Pali Rohár pali.ro...@gmail.com
signature.asc
Description: This is a digitally signed message part.
