[Bug 57950] Replace chained O(U)StringBuffer::append() with operator+

2016-02-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=57950 Commit Notification changed: What|Removed |Added Whiteboard|ToBeReviewed|ToBeReviewed target:5.2.0 -

[Bug 57950] Replace chained O(U)StringBuffer::append() with operator+

2016-02-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=57950 --- Comment #65 from Commit Notification --- Arnold Dumas committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=53bf3ea58e0f4f9f4399a44bfc9ddecc71cb54c7 tdf#5795

[Bug 57950] Replace chained O(U)StringBuffer::append() with operator+

2016-02-11 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=57950 jan iversen changed: What|Removed |Added Whiteboard|target:4.1.0 target:4.2.0 |ToBeReviewed |tar

[Bug 57950] Replace chained O(U)StringBuffer::append() with operator+

2015-11-29 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=57950 --- Comment #63 from Commit Notification --- Sedat Ak committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=1c8e67a9d163e72916d33b5d16211965c9655e23 tdf#57950 Re

[Bug 57950] Replace chained O(U)StringBuffer::append() with operator+

2015-11-29 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=57950 Commit Notification changed: What|Removed |Added Whiteboard|EasyHack SkillCpp |EasyHack SkillCpp

[Bug 57950] Replace chained O(U)StringBuffer::append() with operator+

2015-11-08 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=57950 --- Comment #62 from Commit Notification --- Sedat Ak committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=c4b71b0885122b67e05ed4aed89b0d12ce0f34c8 tdf#57950 Re

[Bug 57950] Replace chained O(U)StringBuffer::append() with operator+

2015-11-08 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=57950 --- Comment #61 from Commit Notification --- Sedat Ak committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=bfc9619fccd17c5059949afc06953695222dea3b tdf#57950 Re

[Bug 57950] Replace chained O(U)StringBuffer::append() with operator+

2015-10-07 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=57950 Commit Notification changed: What|Removed |Added Whiteboard|EasyHack SkillCpp |EasyHack SkillCpp

[Bug 57950] Replace chained O(U)StringBuffer::append() with operator+

2015-10-07 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=57950 --- Comment #60 from Commit Notification --- Aybuke Ozdemir committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=bbfc031333aa74fbd4c6362c33405c49c22283f3 tdf#57

[Bug 57950] Replace chained O(U)StringBuffer::append() with operator+

2015-10-07 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=57950 --- Comment #59 from aybuke --- (In reply to aybuke from comment #58) > I sent following patch for this bug: > > https://gerrit.libreoffice.org/#/c/19027/ I sent accidentally recent reviews. I'm so sorry. I want to send the link patch:

[Bug 57950] Replace chained O(U)StringBuffer::append() with operator+

2015-10-07 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=57950 --- Comment #58 from aybuke --- I sent following patch for this bug: https://gerrit.libreoffice.org/#/c/19027/ -- You are receiving this mail because: You are on the CC list for the bug. ___ L

[Bug 57950] Replace chained O(U)StringBuffer::append() with operator+

2015-10-07 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=57950 --- Comment #57 from aybuke --- I sent following patch for this bug: http://ci.libreoffice.org/job/lo_gerrit_master/7121/ -- You are receiving this mail because: You are on the CC list for the bug. ___

[Bug 57950] Replace chained O(U)StringBuffer::append() with operator+

2015-03-30 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=57950 --- Comment #56 from Commit Notification --- Krisztian Pinter committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=f863d2602fca7d180f49cc4b6fb1bdba57e6a012 fdo#

[Bug 57950] Replace chained O(U)StringBuffer::append() with operator+

2015-03-25 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=57950 --- Comment #55 from Commit Notification --- Austin Chen committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=0fb66458c08b9c0ce59bca85e77d26fad8b59e4b tdf#57950

[Bug 57950] Replace chained O(U)StringBuffer::append() with operator+

2015-03-25 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=57950 Tor Lillqvist changed: What|Removed |Added Assignee|austinche...@gmail.com |libreoffice-b...@lists.free

[Bug 57950] Replace chained O(U)StringBuffer::append() with operator+

2015-03-25 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=57950 Austin Chen changed: What|Removed |Added Assignee|austinchen...@gmail.com |austinche...@gmail.com -- You are

[Bug 57950] Replace chained O(U)StringBuffer::append() with operator+

2015-03-25 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=57950 Austin Chen changed: What|Removed |Added Assignee|libreoffice-b...@lists.free |austinchen...@gmail.com

[Bug 57950] Replace chained O(U)StringBuffer::append() with operator+

2015-03-08 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=57950 --- Comment #52 from mridul --- hey, i would like to contribute to this. can you please suggest me some files where i can find this type of code? -- You are receiving this mail because: You are on the CC list for the bug. ___

[Bug 57950] Replace chained O(U)StringBuffer::append() with operator+

2015-03-08 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=57950 mridul changed: What|Removed |Added Assignee|mridulgup...@gmail.com |libreoffice-b...@lists.free

[Bug 57950] Replace chained O(U)StringBuffer::append() with operator+

2015-03-08 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=57950 mridul changed: What|Removed |Added Assignee|gehadelro...@gmail.com |mridulgup...@gmail.com -- You are recei

[Bug 57950] Replace chained O(U)StringBuffer::append() with operator+

2015-02-23 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=57950 --- Comment #51 from Commit Notification --- Bjoern Michaelsen committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=f8c745d1c060312c03fa020e52acd1337b6e81d9 fdo

[Bug 57950] Replace chained O(U)StringBuffer::append() with operator+

2014-04-17 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=57950 --- Comment #50 from Commit Notification --- Krisztian Pinter committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=9832e23703b989da1dc470f543eb31c25d6c5e29 fdo#57950 R

[Bug 57950] Replace chained O(U)StringBuffer::append() with operator+

2014-03-31 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=57950 --- Comment #49 from Commit Notification --- Krisztian Pinter committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=b5bfa65868b0d1cf735d81686d0f3d97e780c40d fdo#57950 R

[Bug 57950] Replace chained O(U)StringBuffer::append() with operator+

2014-03-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=57950 --- Comment #48 from Commit Notification --- Alexandre Vicenzi committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=5a6c33bdfdf7de03cd0f18d3bba7f4b5749eda70 fdo#57950:

[Bug 57950] Replace chained O(U)StringBuffer::append() with operator+

2014-01-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=57950 Commit Notification changed: What|Removed |Added Whiteboard|EasyHack SkillCpp |EasyHack SkillCpp

[Bug 57950] Replace chained O(U)StringBuffer::append() with operator+

2013-10-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=57950 Qubit changed: What|Removed |Added Whiteboard|EasyHack,SkillCpp,TopicClea |EasyHack SkillCpp |nup,Difficu

[Bug 57950] Replace chained O(U)StringBuffer::append() with operator+

2013-10-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=57950 Björn Michaelsen changed: What|Removed |Added CC||libreoffice@lists.freedeskt

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

2013-04-10 Thread Janit Anjaria
Hey! 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 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

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

2013-04-10 Thread Thorsten Behrens
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 squas

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

2013-04-09 Thread Janit Anjaria
Hey! I am hereby attaching my patch on fdo#57950 where the .append() calls are replaced by + operator. Regards, Janit On Tue, Apr 9, 2013 at 7:00 PM, Janit Anjaria wrote: > Hey Thorsten! > As a mentor it would be great if you review my bug and tell me if i need > to further make any kinda c

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

2013-04-08 Thread Janit Anjaria
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 wrote: > Janit Anjaria wrote: > > I am hereby submitting my patch for review on the bug/feature which > > i had work

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

2013-04-08 Thread Thorsten Behrens
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 O

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

2013-04-08 Thread Janit Anjaria
Hey! 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. Regards, Janit 0001-fdo-57950-replace-chained-O-U-StringBuffer-append-wi.patch Description: Binary data __