<div dir="ltr"><br><br><div class="gmail_quote">On Wed, Aug 6, 2008 at 9:38 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="Ih2E3d">On 06.08.08 04:15:19, Aleix wrote:<br>
> On 8/5/08, Andreas Pakulat <<a href="mailto:apaku@gmx.de">apaku@gmx.de</a>> wrote:<br>
> > On 05.08.08 04:18:56, Aleix wrote:<br>
</div><div class="Ih2E3d">> >  > - This is not very related but needed too, a method to the<br>
> >  > ProjectController to add a custom project in the IDE. I used it to<br>
> >  > instantiate a Project with a custom projectmanager without the need of<br>
> >  > a Plugin and so (even if it is an IPlugin).<br>
> ><br>
> ><br>
> > I don't quite understand why this is needed. Why can't you just load the<br>
> >  project the normal way? IProject isn't really supposed to be subclassed<br>
> >  outside shell/<br>
> I think it is useful to be able to do it without having installed the<br>
> whole plugin and, anyway,<br>
<br>
</div>Which plugin are you talking about? And I still don't understand the<br>
use-case, when would somebody call this function? And why can't he use<br>
openProject.</blockquote><div>Well, it is a code that I can't commit until I have this changes In. <br><br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
<div class="Ih2E3d"><br>
> >  > Index: shell/uicontroller.h<br>
> >  > ===================================================================<br>
> >  > --- shell/uicontroller.h      (revision 842215)<br>
> >  > +++ shell/uicontroller.h      (working copy)<br>
> >  > @@ -40,7 +40,7 @@<br>
> >  >      Q_OBJECT<br>
> >  ><br>
> >  >  public:<br>
> >  > -    UiController(Core *core);<br>
> >  > +    UiController(Core *core, bool ui=true);<br>
> >  >      virtual ~UiController();<br>
> ><br>
> >  Again the flag-thing. Apart from that I don't see the need for this<br>
> >  actually. If you create UiLessController instead of UiController when<br>
> >  doing non-ui Core then there's no need that UiController works without<br>
> >  Ui.<br>
> It is used because the UiLessController inherits UiController (so that<br>
> it works in the Core::initialize())<br>
<br>
</div>I understand the technical need in your patch.<br>
<div class="Ih2E3d"><br>
>  - Abstract the d-pointer. > This makes the implementation a bit ugly I think.<br>
<br>
</div>No, if(d) in all of UiController methods make the implementation quite<br>
ugly - IMHO. A shared dpointer is quite a nice thing actually.<br>
</blockquote><div>Ok, I'll try to do it later today and send another patch version<br><br>Thanks,<br>Aleix<br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
Andreas<br>
<font color="#888888"><br>
--<br>
You will meet an important person who will help you advance professionally.<br>
</font><div><div></div><div class="Wj3C7c"><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></div>