<br><br><div class="gmail_quote">2012/6/13 Jean-Nicolas Artaud <span dir="ltr"><<a href="mailto:jeannicolasartaud@gmail.com" target="_blank">jeannicolasartaud@gmail.com</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br><br><div class="gmail_quote"><div class="im">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>
12.06.2012, 17:59, "Jean-Nicolas Artaud" <<a href="mailto:jeannicolasartaud@gmail.com" target="_blank">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></blockquote></div></div></blockquote><div>Ok, but I have a <span style="background-color:rgb(254,250,223);color:rgb(221,0,0);font-family:Verdana,Arial,Helvetica,sans-serif;font-size:11px;font-weight:bold">Unable to parse diff revision header '(révision 1299895)' !</span></div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="gmail_quote"><div class="im"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div><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>
</div>1. Don't compare boolean return value with 0 - it's incorrect even if old code does that.<br></blockquote></div><div>Ok, fixed. </div><div class="im"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


2. I don't like name of openParseDiffOutput function.</blockquote></div><div>How would you call it. </div><div class="im"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Also it seems that it mixes GUI stuff and parsing<br>

(probably show() should stay where it was)<br></blockquote></div><div>It means youhave to change the openDiff too, I don't see the interest. </div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


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