<br><br><div class="gmail_quote">2012/6/13 Konstantin Tokarev <span dir="ltr"><<a href="mailto:annulen@yandex.ru" target="_blank">annulen@yandex.ru</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
13.06.2012, 16:11, "Jean-Nicolas Artaud" <<a href="mailto:jeannicolasartaud@gmail.com">jeannicolasartaud@gmail.com</a>>:<br>
<div class="im">> 2012/6/13 Konstantin Tokarev <<a href="mailto:annulen@yandex.ru">annulen@yandex.ru</a>><br>
>> 12.06.2012, 17:59, "Jean-Nicolas Artaud" <<a href="mailto:jeannicolasartaud@gmail.com">jeannicolasartaud@gmail.com</a>>:<br>
>>> Hello,Sorry but I didn't succeed with the review board, only know the git one :-(<br>
>><br>
>> <a href="http://techbase.kde.org/Development/Review_Board#Using_Review_Board_With_Subversion" target="_blank">http://techbase.kde.org/Development/Review_Board#Using_Review_Board_With_Subversion</a><br>
>><br>
>> It's possible to submit git-originated patches there too, but they need to be modifed to look like<br>
>> svn diffs<br>
>><br>
>>> BTW, I have a patch to fix the bug 137826.<br>
>>> Could you please review it and/or guide me a bit more to use the review board ?<br>
>><br>
>> 1. Don't compare boolean return value with 0 - it's incorrect even if old code does that.<br>
> Ok, fixed.<br>
>> 2. I don't like name of openParseDiffOutput function.<br>
> How would you call it.<br>
<br>
</div>parseAndOpenDiff<br></blockquote><div><br></div><div>Ok, really nice, I'll put this in the next patch :-)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<span class="HOEnZb"><font color="#888888"><br>
--<br>
Regards,<br>
Konstantin<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div>Jean-Nicolas</div><br>