<div dir="ltr"><div><div>Hey!<br></div>i am here by attaching my patch for fdo#62096.<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:47 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 Wednesday, 2013-04-17 00:00:39 +0530, Janit Anjaria wrote:<br>
<br>
> -            if ( sInstModel.compareTo( pProps->Name ) == 0 )<br>
> +            if ( sInstModel == ( pProps->Name ) )<br>
<br>
I'd prefer if you removed also the now extraneous parenthesis<br>
additionally to the .compareTo, so that would look like<br>
<br>
               if ( sInstModel == pProps->Name )<br>
<br>
It's just easier to parse for the eye ;-)  Maybe it's just me, but<br>
I feel tricked into "that's a function lacking the function name" ...<br>
<br>
And please, use fdo#62096 instead of BUG#62096<br>
<br>
Thanks<br>
<span class="HOEnZb"><font color="#888888">  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>