[Libreoffice] [PATCH][PUSHED] Duplicate code: join ImportFrom and InsertFrom

Petr Mladek pmladek at suse.cz
Mon Jun 6 10:53:00 PDT 2011


Noel Power píše v St 01. 06. 2011 v 10:16 +0100:
> On 31/05/11 13:03, Chr. Rossmanith wrote:
> > Hi,
> >
> > more duplicate code cleanup.
> >
> > Class SfxObjectShell has two nearly identical methods: ImportFrom and 
> > InsertFrom. The latter has a few lines of code more, so I've removed 
> > InsertFrom (which was added to the code base later than ImportFrom), 
> > added a boolean parameter to ImportFrom and adjusted the few calls to 
> > those methods. And InsertFrom is not virtual like ImportFrom. Please 
> > review the attached patches. I'll commit them if I get an "ok".
> looks good to me, please commit it to master ( or I will commit it later 
> after I get a build /me unfortunately accidently did a make clean on his 
> master build )

I see all patches pushed in master now.

Thanks for the nice work.


Best Regards,
Petr



More information about the LibreOffice mailing list