<div dir="ltr"><div><div><div><div>Hey!<br><br></div>I am hereby attaching my patch for fdo #62096 :<br><br><a href="https://bugs.freedesktop.org/show_bug.cgi?id=62096" target="_blank">https://bugs.freedesktop.org/show_bug.cgi?id=62096</a><br>

<br></div>Any suggestions would be great to be implemented.<br><br></div>Regards,<br></div>Janit<div class="gmail_extra"><br><div class="gmail_quote">On Tue, Apr 16, 2013 at 11:52 PM, Janit Anjaria <span dir="ltr"><<a href="mailto:janit92@gmail.com" target="_blank">janit92@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><div><div><div>Hey Stephan!<br></div>I think that was due to hurry(the things for "!=" things)...and yeah i dont know ,i overlooked the build and hence i suppose the thing you just mentioned about child_process.cxx is one of those rotten tests.<br>


<br></div>And yes,ill surely look into the matter with identation(i generally do take care of it,but because of a lil hurry as i am also working on the project -i had mailed on the list simultaneously ...so swamped with too much of code ! :P )<br>


<br></div><div>Hope this finds you in good health.<br><br></div>Regards,<br></div>Janit<br></div><div><div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Apr 16, 2013 at 6:30 PM, Stephan Bergmann <span dir="ltr"><<a href="mailto:sbergman@redhat.com" target="_blank">sbergman@redhat.com</a>></span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div>On 04/15/2013 03:19 PM, Janit Anjaria wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
I am hereby attaching my patch for fdo #62096 :<br>
<br>
<a href="https://bugs.freedesktop.org/show_bug.cgi?id=62096" target="_blank">https://bugs.freedesktop.org/<u></u>show_bug.cgi?id=62096</a><br>
<br>
Any suggestions would be great to be implemented.<br>
</blockquote>
<br></div>
First, note that<br>
<br>
  s.compareTo(t) != 0<br>
<br>
is /not/ the same as<br>
<br>
  s == t<br>
<br>
Second, did you build and test your change?  It looks like sal/qa/rtl/process/child_<u></u>process.cxx is one of those rotting tests that is not built nor executed, not even with a top-level "make check".<br>
<br>
Also, please take care not to mess up the indentation of lines, making the "if" indented only three instead of four spaces.<span><font color="#888888"><br>
<br>
Stephan<br>
</font></span></blockquote></div><br></div>
</div></div></blockquote></div><br></div></div>