REMINDER: Final review of VCS interfaces

[Robert Gruber]

On Thu, 24 May 2007 22:38:38 Andreas Pakulat wrote:
> On 24.05.07 22:10:34, Robert Gruber wrote:
> > I've planned to do the porting of the CVS plugin. I just didn't start 
> > yet, because the interface was still changing a lot. As soon as we 
> > decide that the interface is good enough to give it a try, I could 
> > start implementing it almost right away.
> > But I have no objections if want to do the porting.
> 
> Ok, thats nice. I just "volunteered" because Matt was kind of
> complaining and I had something in the back of my head that you don't
> have much free time to hack on cvs plugin. So if you have the time
> please port it yourself, as I said I'm not that familiar with cvs
> anymore and I don't know the code so you'd be faster done.

I tried to start implementing the new IBasicVersionControl interface in the CVS plugin. But got stopped at the very beginning. This was already issued some time ago, but somehow nothing happened. All methods return a VcsJob object, but they return it as value. Wouldn't be returning a pointer be better? 


I'm still not sure about returning VcsJob anyways. For example in cvs plugin I'm using my own CvsJob class which is derived from KJob and therefore already provides all that stuff from VcsJob. So why don't we just return a KJob pointer? Or maybe derive VcsJob from KJob, which will make the start and cancel slots and the signals needless as they are already provided by KJob.


Robert

-- 
Ist Ihr Browser Vista-kompatibel? Jetzt die neuesten 
Browser-Versionen downloaden: http://www.gmx.net/de/go/browser