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

Janit Anjaria janit92 at gmail.com
Mon Apr 8 19:36:56 PDT 2013


Hey Thorsten,
I am not pretty sure how can that be possible but yes i would clean the
patch up and then mail it .

Regards,
Janit


On Tue, Apr 9, 2013 at 4:02 AM, Thorsten Behrens <thb at documentfoundation.org
> wrote:

> Janit Anjaria wrote:
> > I am hereby submitting my patch for review on the bug/feature which
> > i had worked long back which replaces all the .append() calls witha
> > + operator.
> >
> Hi Janit,
>
> thanks for that patch - it needs some cleanup, too, though. String
> literals can be appended without explicit OUString constructor
> calls. And why is there code that is now commented out?
>
> Cheers,
>
> -- Thorsten
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/libreoffice/attachments/20130409/7bd340ad/attachment-0001.html>


More information about the LibreOffice mailing list