[Libreoffice] [PATCH] Duplicate code: join ImportFrom and InsertFrom
Christina Rossmanith
ChrRossmanith at gmx.de
Wed Jun 1 03:27:31 PDT 2011
Yesterday everything built fine. Then I did a "./g pull -r" which caused the build to fail. As soon as I have a successfull build again I'll push everything which has queued up until now :-)
Christina
-------- Original-Nachricht --------
> Datum: Wed, 01 Jun 2011 10:16:54 +0100
> Von: Noel Power <nopower at novell.com>
> An: "Chr. Rossmanith" <ChrRossmanith at gmx.de>
> CC: libreoffice at lists.freedesktop.org
> Betreff: Re: [Libreoffice] [PATCH] Duplicate code: join ImportFrom and InsertFrom
> 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
> _______________________________________________
> LibreOffice mailing list
> LibreOffice at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/libreoffice
--
Christina Rossmanith
Empfehlen Sie GMX DSL Ihren Freunden und Bekannten und wir
belohnen Sie mit bis zu 50,- Euro! https://freundschaftswerbung.gmx.de
More information about the LibreOffice
mailing list