[PATCH] BUG#62096 Replace O(U)String compareTo with ==
Eike Rathke
erack at redhat.com
Fri Apr 19 12:17:05 PDT 2013
Hi Janit,
On Wednesday, 2013-04-17 00:00:39 +0530, Janit Anjaria wrote:
> - if ( sInstModel.compareTo( pProps->Name ) == 0 )
> + if ( sInstModel == ( pProps->Name ) )
I'd prefer if you removed also the now extraneous parenthesis
additionally to the .compareTo, so that would look like
if ( sInstModel == pProps->Name )
It's just easier to parse for the eye ;-) Maybe it's just me, but
I feel tricked into "that's a function lacking the function name" ...
And please, use fdo#62096 instead of BUG#62096
Thanks
Eike
--
LibreOffice Calc developer. Number formatter stricken i18n transpositionizer.
GPG key ID: 0x65632D3A - 2265 D7F3 A7B0 95CC 3918 630B 6A6C D5B7 6563 2D3A
For key transition see http://erack.de/key-transition-2013-01-10.txt.asc
Support the FSFE, care about Free Software! https://fsfe.org/support/?erack
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/libreoffice/attachments/20130419/12f86bd4/attachment.pgp>
More information about the LibreOffice
mailing list