From koffice-devel  Wed Jun 17 18:42:51 2009
From: Boudewijn Rempt <boud () valdyas ! org>
Date: Wed, 17 Jun 2009 18:42:51 +0000
To: koffice-devel
Subject: Re: koffice/libs/flake [POSSIBLY UNSAFE]
Message-Id: <200906172042.51570.boud () valdyas ! org>
X-MARC-Message: https://marc.info/?l=koffice-devel&m=124526415120636

On Wednesday 17 June 2009, Thorsten Zachmann wrote:
> On Wed June 17 2009, you wrote:
> > On Wed, 17 Jun 2009, Thorsten Zachmann wrote:
> > > Hello,
> > >
> > > > Note: I moved KoRTree from KoMain to flake, since it was used by
> > > > flake and kspread, and kspread uses flake anyway.
> > >
> > > maybe KoRTree can be moved to a place where it is easier for other apps
> > > (outside of koffice) to use. As far as I remember it should not have
> > > any dependencies into other parts of koffice.
> >
> > I considered putting it in a lib all of its own :-). Then I considered
> > KoBase, but that is depenedent on KoStore. I'm not sure what the right
> > solution is, but I'm open to any hints.
>
> how about removing the only dependency on KoStore which is
>
> void KoUnit::saveOdf(KoXmlWriter* settingsWriter, KoUnit _unit)
> {
>     settingsWriter->addConfigItem("unit", unitName(_unit));
> }
>
> if I'm not mistaken. We could move this function to a helper function into
> koodf. If we can remove this dependency KoBase is no longer dependent on
> kostore and it is easy to move stuff like KoRTree into it.

Good plan! Can you commit the change?


-- 
Boudewijn Rempt | http://www.valdyas.org
_______________________________________________
koffice-devel mailing list
koffice-devel@kde.org
https://mail.kde.org/mailman/listinfo/koffice-devel