<div dir="ltr">Hey!<div>I aam sure the mentor at that point in time initially when i started my work on the LO codebase...he asked me to do the same ,and that is why i made the following changes....</div><div>But yeah if you still feel i need to change the patch according with the buffer thing resolved,ill do it and mail it to you asap.</div>
<div><br></div><div>:)</div><div><br></div><div>Regards,</div><div>Janit<br><div><br></div></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Apr 10, 2013 at 3:22 PM, Thorsten Behrens <span dir="ltr"><<a href="mailto:thb@documentfoundation.org" target="_blank">thb@documentfoundation.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">Janit Anjaria wrote:<br>
> I am hereby attaching my patch on fdo#57950 where the .append() calls are<br>
> replaced by  + operator.<br>
><br>
</div>Hi Janit,<br>
<br>
this does not seem to be a patch against master - the code there has<br>
OUStringBuffer sBuffer etc etc, not OUString sBuffer as required by<br>
your patch. Please squash your various changes you did to that file<br>
into one commit, see e.g.<br>
<a href="http://gitready.com/advanced/2009/02/10/squashing-commits-with-rebase.html" target="_blank">http://gitready.com/advanced/2009/02/10/squashing-commits-with-rebase.html</a><br>
<br>
Cheers,<br>
<br>
-- Thorsten<br>
</blockquote></div><br></div>