Bug 137826 - "next file" button does not work when used first thing in a new session

Konstantin Tokarev annulen at yandex.ru
Wed Jun 13 10:51:58 UTC 2012


12.06.2012, 17:59, "Jean-Nicolas Artaud" <jeannicolasartaud at gmail.com>:
> Hello,Sorry but I didn't succeed with the review board, only know the git one :-(

http://techbase.kde.org/Development/Review_Board#Using_Review_Board_With_Subversion

It's possible to submit git-originated patches there too, but they need to be modifed to look like
svn diffs

> BTW, I have a patch to fix the bug 137826.
> Could you please review it and/or guide me a bit more to use the review board ?

1. Don't compare boolean return value with 0 - it's incorrect even if old code does that.
2. I don't like name of openParseDiffOutput function. Also it seems that it mixes GUI stuff and parsing
(probably show() should stay where it was)


-- 
Regards,
Konstantin


More information about the Kompare-devel mailing list