[PUSHED] Remove unused code

2013-04-18 Thread David Tardon (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/3447 Approvals: David Tardon: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/3447 To unsubscribe, visit

[PUSHED] remove unused code svtools/bmpmaker

2013-03-21 Thread Noel Power (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/2868 Approvals: Noel Power: Verified; Looks good to me, approved LibreOffice gerrit bot: Verified Thomas Arnhold: Verified; Looks good to me,

[PUSHED] Remove unused code

2012-09-24 Thread Noel Power (via Code Review)
Hi, Thank you for your patch! :-) It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/682 Approvals: Noel Power: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/682 To unsubscribe, visit

Re: [PUSHED] Remove unused code (oox)

2012-04-13 Thread Miklos Vajna
Hi Santiago, On Thu, Apr 12, 2012 at 08:04:19PM +0200, Santiago Martinez wrote: > This patch removes unused code as listed in unusedcode.easy Thanks for this. Given that this code used to be in oox, it still lives in the oox namespace, though I think it makes more sense to mention the real mod

Re: [PATCH][PUSHED] remove unused code (oox, sd)

2012-03-15 Thread Cedric Bosdonnat
Hi Petr, On Thu, 2012-03-15 at 12:45 +0100, Petr Vorel wrote: > remove some unused code in oox and sd. Thanks for your patch! I just pushed it to master branch. Keep sending good patches like this one ;) -- Cedric ___ LibreOffice mailing list LibreOff

Re: [PATCH] [PUSHED] Remove unused code in dpitemdata

2012-03-12 Thread Kohei Yoshida
Pushed this one also. Thanks a lot. I was afraid some of your patches might cause conflicts (since these are the areas I was modifying a lot in the past few weeks), but luckily all your patches applied cleanly. Regards, Kohei ___ LibreOffice mailing

Re: [PATCH] [PUSHED] Remove unused code in dptabres

2012-03-12 Thread Kohei Yoshida
Pushed to master. Thanks a lot. Kohei ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice

Re: [PATCH] [PUSHED] Remove unused code

2012-03-12 Thread Kohei Yoshida
Pushed to master. Kohei ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice

Re: [PATCH] [PUSHED] Remove unused code (23)

2012-03-05 Thread Tor Lillqvist
Thanks, pushed to master. --tml ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice

Re: [PATCH] [PUSHED] Remove unused code (22)

2012-03-05 Thread Tor Lillqvist
Thanks, pushed to master. --tml ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice

Re: [PATCH] [PUSHED] remove unused code

2012-03-05 Thread Tor Lillqvist
Pushed to master, thanks! --tml ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice

Re: [PATCH] [PUSHED] Remove unused code (21)

2012-03-01 Thread Tor Lillqvist
Thanks, pushed to master. --tml ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice

Re: [PUSHED] Remove unused code (19)

2012-02-26 Thread Ivan Timofeev
Hi Elton, On 26.02.2012 14:42, Elton Chung wrote: Sorry, please revert the removal of line 416-500 in the original patch which I overlooked. http://cgit.freedesktop.org/libreoffice/core/commit/?id=0499a44cddae31dff5ce1959aba8aa64119dea90 Cheers, Ivan __

Re: [PUSHED] Remove unused code (19)

2012-02-26 Thread Elton Chung
Hi Michael,*, Sorry, please revert the removal of line 416-500 in the original patch which I overlooked. Best Regards, Elton 2012/2/26, Michael Meeks : > > On Sat, 2012-02-25 at 01:57 +0800, Elton Chung wrote: >> This patch removes 14 unused methods. > > Thanks, pushed - modulo the fround r

[PUSHED] Remove unused code (20)

2012-02-25 Thread Michael Meeks
On Sat, 2012-02-25 at 14:15 +0800, Elton Chung wrote: > This patch removes 20 unused methods. Thanks ! :-) Michael. -- michael.me...@suse.com <><, Pseudo Engineer, itinerant idiot ___ LibreOffice mailing list LibreOffice@lis

[PUSHED] Remove unused code (19)

2012-02-25 Thread Michael Meeks
On Sat, 2012-02-25 at 01:57 +0800, Elton Chung wrote: > This patch removes 14 unused methods. Thanks, pushed - modulo the fround removal which we added a unit test for IIRC. :-) Michael. -- michael.me...@suse.com <><, Pseudo Engineer, itinerant idiot

[PUSHED] Remove unused code in ScUserList

2012-02-23 Thread Michael Meeks
On Wed, 2012-02-22 at 21:40 +0100, Santiago Martinez wrote: > This patch removes unused code as listed in unusedcode.easy Thanks, pushed :-) Michael. -- michael.me...@suse.com <><, Pseudo Engineer, itinerant idiot ___ Libre

Re: [PUSHED] Remove unused code (18)

2012-02-20 Thread Elton Chung
Hi Michael, *, >        Thanks for that ! :-) are you tempted by any of the Easy Hacks ? I will try them (and ask for some help) when I am more familiar with the code base. Best Regards, Elton -- Elton Chung Administrator, Sponsor of mirror.layerjet.com Language Maintainer, Tester of ReactOS P

[PUSHED] Remove unused code (18)

2012-02-20 Thread Michael Meeks
On Mon, 2012-02-20 at 21:25 +0800, Elton Chung wrote: > This patch removes another 16 unused methods. Thanks for that ! :-) are you tempted by any of the Easy Hacks ? All the best, Michael. -- michael.me...@suse.com <><, Pseudo Engineer, itinerant idiot _

[PUSHED] Remove unused code (17)

2012-02-20 Thread Michael Meeks
On Mon, 2012-02-20 at 00:33 +0800, Elton Chung wrote: > This patch removes 14 unused methods from basegfx. Pushed; thanks :-) Michael. -- michael.me...@suse.com <><, Pseudo Engineer, itinerant idiot ___ LibreOffice mailing l

[PUSHED] Remove unused code in XMLTextListAutoStylePool

2012-02-20 Thread Michael Meeks
On Sun, 2012-02-19 at 19:04 +0100, Santiago Martinez wrote: > This patch removes unused code as listed in unusedcode.easy Pushed, thanks :-) Michael. -- michael.me...@suse.com <><, Pseudo Engineer, itinerant idiot ___ LibreO

[PUSHED] Remove unused code

2012-02-20 Thread Michael Meeks
On Mon, 2012-02-20 at 01:13 +0100, Petr Vorel wrote: > another bit of unused core removed. Thanks for that, pushed :-) Michael. -- michael.me...@suse.com <><, Pseudo Engineer, itinerant idiot ___ LibreOffice mailing list Lib

Re: [PATCH][PUSHED] Remove unused code in XclRoot

2012-02-19 Thread Markus Mohrhard
Makes sense. Pushed. Thanks a lot for this patch. 2012/2/19 Santiago Martinez : > This patch removes unused code as listed in unusedcode.easy > ___ > LibreOffice mailing list > LibreOffice@lists.freedesktop.org > http://lists.freedesktop.org/mailman/lis

Re: [PATCH][PUSHED] Remove unused code in XclExpName

2012-02-19 Thread Markus Mohrhard
Makes sense. Pushed. Thanks a lot for the patch. 2012/2/19 Santiago Martinez : > This patch removes unused code as listed in unusedcode.easy > > > ___ > LibreOffice mailing list > LibreOffice@lists.freedesktop.org > http://lists.freedesktop.org/mailman/

[PUSHED] Remove unused code (13)

2012-02-18 Thread Michael Meeks
Hi Helton, On Sat, 2012-02-18 at 14:34 +0800, Elton Chung wrote: > This patch removes 9 unused methods. Thanks for that :-) Michael. -- michael.me...@suse.com <><, Pseudo Engineer, itinerant idiot ___ LibreOffice mailing lis

Re: [PUSHED] Remove unused code in core/svx/source/dialog/_contdlg.cxx

2012-02-17 Thread Thorsten Behrens
Thanks a lot, pushed. Keep 'em coming, -- Thorsten pgpiLVIhoD1ix.pgp Description: PGP signature ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice

[PUSHED] Remove unused code in SvxSearchDialog

2012-02-17 Thread Michael Meeks
Hi there, On Thu, 2012-02-16 at 21:39 +0100, Santiago Martinez wrote: > This patch removes unused code as listed in unusedcode.easy Thanks - pushed :-) Michael. -- michael.me...@suse.com <><, Pseudo Engineer, itinerant idiot ___

Re: [PUSHED] Remove unused code (8)

2012-02-13 Thread Ivan Timofeev
Hi Elton, Michael, On 13.02.2012 20:04, Michael Meeks wrote: On Sat, 2012-02-11 at 00:50 +0800, Elton Chung wrote: This patch removes 10 unused methods. Lovely; if you 'git grep bNetLock' - I think we can remove that item from the header, and it's remaining initialization too now (cou

[PUSHED] Remove unused code (12)

2012-02-13 Thread Michael Meeks
On Sun, 2012-02-12 at 17:10 +0800, Elton Chung wrote: > This patch removes 3 unused methods. Pushed ! :-) and that was the last queued up patch from you I think, thanks for all the great work :-) All the best, Michael. -- michael.me...@suse.com <><, Pseudo Engi

[PUSHED] Remove unused code (11)

2012-02-13 Thread Michael Meeks
On Sun, 2012-02-12 at 17:09 +0800, Elton Chung wrote: > This patch removes 12 unused methods. Pushed with some conflicts removed, Thanks, Michael. -- michael.me...@suse.com <><, Pseudo Engineer, itinerant idiot ___

[PUSHED] Remove unused code (10)

2012-02-13 Thread Michael Meeks
On Sun, 2012-02-12 at 17:07 +0800, Elton Chung wrote: > This patch removes 8 unused methods. Thanks :-) Michael. -- michael.me...@suse.com <><, Pseudo Engineer, itinerant idiot ___ LibreOffice mailing list LibreOffice@lists.f

[PUSHED] Remove unused code (8)

2012-02-13 Thread Michael Meeks
On Sat, 2012-02-11 at 00:50 +0800, Elton Chung wrote: > This patch removes 10 unused methods. Lovely; if you 'git grep bNetLock' - I think we can remove that item from the header, and it's remaining initialization too now (could you hack that up) ? :-) Similarly for bNotMasterCach

[PUSHED] Remove unused code (7)

2012-02-13 Thread Michael Meeks
On Sat, 2012-02-11 at 00:49 +0800, Elton Chung wrote: > This patch removes another 4 unused methods. Thanks :-) Michael. -- michael.me...@suse.com <><, Pseudo Engineer, itinerant idiot ___ LibreOffice mailing list LibreOffic

[PUSHED] Remove unused code (6)

2012-02-13 Thread Michael Meeks
On Sat, 2012-02-11 at 00:48 +0800, Elton Chung wrote: > The patch removes 4 unused methods. Pushed; thanks :-) Michael. -- michael.me...@suse.com <><, Pseudo Engineer, itinerant idiot ___ LibreOffice mailing list LibreOffice@

Re: [PATCH][PUSHED] Remove unused code (5)

2012-02-13 Thread Stephan Bergmann
On 02/11/2012 03:59 PM, Ivan Timofeev wrote: Pushed, thank you. marking thread as pushed Stephan ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice

Re: [Libreoffice] [PATCH][PUSHED] Remove unused code

2012-01-02 Thread Lubos Lunak
On Wednesday 28 of December 2011, Marcel Metz wrote: > Hello lo-devs, > > There is never an instance of mpDateTable allocated and a lot of > code is never executed because of that. This patch removes the > unused code. Pushed, thanks. -- Lubos Lunak l.lu...@suse.cz ___