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

Noel Power nopower at novell.com
Wed Jun 1 02:16:54 PDT 2011


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 )

thanks
Noel


More information about the LibreOffice mailing list