On a general note, I'm pretty much a supporter of the line "we need to keep KoShape as slim as up to nearly impossible". So anything which on the one hand is not of a complete universal use and on the other hand could be implemented outside should in my opinion be left outside.<br>

<br>PierreSt<br><br><br><div class="gmail_quote">On Tue, Jun 28, 2011 at 6:00 AM, Thorsten Zachmann <span dir="ltr"><<a href="mailto:t.zachmann@zagge.de">t.zachmann@zagge.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

<div class="im">On Monday, June 27, 2011 22:07:28 <a href="mailto:jaham@gmx.net">jaham@gmx.net</a> wrote:<br>
> On Monday 27 June 2011 17:25:37 Thorsten Zachmann wrote:<br>
> > On Sunday, June 26, 2011 18:25:56 <a href="mailto:jaham@gmx.net">jaham@gmx.net</a> wrote:<br>
> > > I thought to have a separate lib similar to our odf lib containing the<br>
> > > classes  used for loading/saving svg. One of these classes would be an<br>
> > > interface which shapes can implement to save/load svg data, i.e.<br>
> > > calling it SvgSerializable.<br>
> > ><br>
> > > class SvgSerializable<br>
> > > {<br>
> > ><br>
> > > public:<br>
> > >         virtual bool saveSvg(SvgSavingContext &context) = 0;<br>
> > >         virtual bool loadSvg(SvgLoadingContext &context) = 0;<br>
> > >         // some more required functions<br>
> > ><br>
> > > };<br>
> ><br>
> > Agreed that having a libs like the odf lib makes sense and that contains<br>
> > the stuff like SvgLoadingContext .... However I'm not sure if we need a<br>
> > class SvgSerializable. Why not just put that into KoShape directly?<br>
><br>
> There is no reason other than I was a little hesistant to force svg<br>
> loading/saving support on each shape. Instead I thought making it<br>
> optionally. But if there is consent about putting it directly into<br>
> KoShape, I have no problem with that.<br>
<br>
</div>Looks like I did not understand how you want to use the above. For me it<br>
looked like KoShape would need to inherit SvgSerializable to be able to<br>
implement the functions. Seems I was wrong. Maybe you can explain how that is<br>
should be done.<br>
<font color="#888888"><br>
Thorsten<br>
</font><div><div></div><div class="h5"><br>
_______________________________________________<br>
calligra-devel mailing list<br>
<a href="mailto:calligra-devel@kde.org">calligra-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/calligra-devel" target="_blank">https://mail.kde.org/mailman/listinfo/calligra-devel</a><br>
</div></div></blockquote></div><br>