Adding additional IOutputViewModel

dukju ahn dukjuahn at gmail.com
Tue Jun 12 18:17:36 UTC 2007


2007/6/11, Andreas Pakulat <apaku at gmx.de>:
> On 11.06.07 15:21:14, dukju ahn wrote:
> > You are right. We use IOutputViewModel. We don't need OutputViewModel.
>
> Would you please put the IOutputViewModel into a separate header and
> implementation. Its not obvious that the interface that is needed is
> inside the outputmodel header. Also not all plugins may want to use the
> standard outputmodel...
>
> > I attatched real working code patch. I didn't commit yet because it assumes
> > that there's only one view, and current tab is an active one, the temporary
> > assumptions that will be addressed eventually.
>
> I think you can commit this (unless it doesn't compile), after fixing
> the following things (the move mentioned above). I'll add the toolbutton
> for now to make the activation thing work afterwards.

Every comments from you were applied, and committed.
Thanks for reviewing.




More information about the KDevelop-devel mailing list