[PATCH]BUG#57950 Replace chained O(U)StringBuffer::append() with operator+

Thorsten Behrens thb at documentfoundation.org
Wed Apr 10 02:52:32 PDT 2013


Janit Anjaria wrote:
> I am hereby attaching my patch on fdo#57950 where the .append() calls are
> replaced by  + operator.
> 
Hi Janit,

this does not seem to be a patch against master - the code there has
OUStringBuffer sBuffer etc etc, not OUString sBuffer as required by
your patch. Please squash your various changes you did to that file
into one commit, see e.g.
http://gitready.com/advanced/2009/02/10/squashing-commits-with-rebase.html

Cheers,

-- Thorsten
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.freedesktop.org/archives/libreoffice/attachments/20130410/ac3aa853/attachment.pgp>


More information about the LibreOffice mailing list