VCS Interfaces, round 3

[Andreas Pakulat]

On 04.05.07 14:51:11, Matthew Woehlke wrote:
> Andreas Pakulat wrote:
> > On 04.05.07 14:07:39, Matthew Woehlke wrote:
> >> Andreas Pakulat wrote:
> >>> On 04.05.07 10:33:08, Matthew Woehlke wrote:
> >>>> Matt Rogers wrote:
> >>>>> Why do we have one interface that takes local paths and one that  
> >>>>> takes repo paths?
> >>>> Ask Andreas. :-) Personally I'd be in favor of dropping the one in 
> >>>> IBasicVC (that takes local paths), you can always still use local paths 
> >>>> indirectly with repositoryLocation().
> >>> Uhm, actually I think I was a bit confused. Add certainly doesn't make
> >>> the slightest sense to have in the repo interface. But remove does.
> >> I think this conversation just wandered off and got lost. I was asking 
> >> about two instances of log() (at least, that's what I remember thinking 
> >> I was doing? ;-)), although you're right, IRepoVC should have a remove().
> > 
> > Hmm, I can't recall any discussion about removing add() from the RepoVC.
> > remove() is there already (last time I looked at least).
> 
> We're talking about 
> svn.kde.org/home/kde/trunk/KDE/kdevelop/lib/plugins/vcs/interfaces/irepositoryversioncontrol.h 
> right?

I guess my brain needs a break. You're absolutely right.

Andreas

-- 
Reply hazy, ask again later.