<div class="gmail_quote">On Mon, Aug 10, 2009 at 12:23 AM, Andreas Pakulat <span dir="ltr"><<a href="mailto:apaku@gmx.de">apaku@gmx.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="im">On 09.08.09 23:44:42, Andreas Pakulat wrote:<br>
> On 09.08.09 21:30:02, Aleix Pol wrote:<br>
> > Hi,<br>
> > Do you think it would be possible to move the PartDocument somewhere else<br>
> > (like in kdevplatform/util) so that we can create new document instances<br>
> > with it?<br>
><br>
> No, rather it should move to interfaces IMHO, thats basically the<br>
> "public API" of shell. One downside is that this means interfaces links<br>
> against sublime and hence each plugin links against sublime too.<br>
><br>
> That would (hopefully) also allow to remove some duplicated code between<br>
> PartDocument, TextDocument and DesignerDocument (currently in<br>
> playground)<br>
<br>
</div>You also might hit problems where PartDocument uses shell-internal API.<br>
Please try to keep as much of that inside shell as possible...<br>
<br>
Hmm, is there any reason not to implement a KDevKompareDocument directly<br>
inside shell, instead of in a plugin? Because I'm not overly comfortable<br>
to create new public API for just 1 specific case, that'll need quite<br>
some thinking on what is needed also in other cases<br>
<br>
Andreas<br>
<font color="#888888"><br>
--<br>
You love peace.<br>
</font><div><div></div><div class="h5"><br>
_______________________________________________<br>
KDevelop-devel mailing list<br>
<a href="mailto:KDevelop-devel@kdevelop.org">KDevelop-devel@kdevelop.org</a><br>
<a href="https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel" target="_blank">https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel</a><br>
</div></div></blockquote></div><br>Yes, that's what I ended up by doing. I moved the PatchDocument to the shell and created a IPatchDocument as an interface and seems to work just fine.<br><br>Now i'm getting some ghost crash that's driving me mad... gaaah xD<br>