<div dir="ltr"><div><div>Hey!<br></div>I am hereby submitting my patches for review.<br><br><br></div>Regards,<br>Janit<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sat, Apr 20, 2013 at 12:53 AM, Eike Rathke <span dir="ltr"><<a href="mailto:erack@redhat.com" target="_blank">erack@redhat.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Janit,<br>
<br>
On Friday, 2013-04-19 02:09:55 +0530, Janit Anjaria wrote:<br>
<br>
> -    if ( 0 == PropertyName.compareTo( FM_PROP_TEXTLINECOLOR ) )<br>
> +    if ( PropertyName == ( FM_PROP_TEXTLINECOLOR ) )<br>
<br>
Same nitpick as in my previous mail, please remove the now extraneous ()<br>
<span class="HOEnZb"><font color="#888888"><br>
  Eike<br>
<br>
--<br>
LibreOffice Calc developer. Number formatter stricken i18n transpositionizer.<br>
GPG key ID: 0x65632D3A - 2265 D7F3 A7B0 95CC 3918  630B 6A6C D5B7 6563 2D3A<br>
For key transition see <a href="http://erack.de/key-transition-2013-01-10.txt.asc" target="_blank">http://erack.de/key-transition-2013-01-10.txt.asc</a><br>
Support the FSFE, care about Free Software! <a href="https://fsfe.org/support/?erack" target="_blank">https://fsfe.org/support/?erack</a><br>
</font></span></blockquote></div><br></div>