Review Request: Simplified IBasicVersionControl::diff

Fabian Wiesel fabian.wiesel at googlemail.com
Mon Mar 23 18:55:14 UTC 2009



> On 2009-03-23 11:30:14, Andreas Pakulat wrote:
> > /trunk/KDE/kdevplatform/vcs/interfaces/ibasicversioncontrol.h, line 163
> > <http://reviewboard.kde.org/r/395/diff/2/?file=3635#file3635line163>
> >
> >     These namespace changes seem to be unrelated and I don't see the need for them.

You mixed up the old and the new version. It removes the unneeded scope prefix.


> On 2009-03-23 11:30:14, Andreas Pakulat wrote:
> > /trunk/KDE/kdevplatform/vcs/interfaces/ibasicversioncontrol.h, line 170
> > <http://reviewboard.kde.org/r/395/diff/2/?file=3635#file3635line170>
> >
> >     Didn't we agree that we should use a plain Url?

And so it is.


- Fabian


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/395/#review572
-----------------------------------------------------------


On 2009-03-22 10:43:38, Fabian Wiesel wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/395/
> -----------------------------------------------------------
> 
> (Updated 2009-03-22 10:43:38)
> 
> 
> Review request for KDevelop.
> 
> 
> Summary
> -------
> 
> The current IBasicVersionControl::diff interface seems to be too powerful for most VCSs. Replaced it with a simpler one, only allowing diffs between different versions of the same file.
> Moved the old one to Subversion-Plugin, as it is the only one implementing it.
> 
> Did the same with merge() as it requires the same capabilities of the VCS as diff().
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdevplatform/plugins/cvs/cvsplugin.h 942782 
>   /trunk/KDE/kdevplatform/plugins/cvs/cvsplugin.cpp 942782 
>   /trunk/KDE/kdevplatform/plugins/mercurial/mercurialplugin.h 942782 
>   /trunk/KDE/kdevplatform/plugins/mercurial/mercurialplugin.cpp 942782 
>   /trunk/KDE/kdevplatform/plugins/subversion/kdevsvnplugin.h 942782 
>   /trunk/KDE/kdevplatform/plugins/subversion/kdevsvnplugin.cpp 942782 
>   /trunk/KDE/kdevplatform/plugins/vcscommon/kdevvcscommonplugin.cpp 942782 
>   /trunk/KDE/kdevplatform/shell/kross/krossdistributedversioncontrol.h 942782 
>   /trunk/KDE/kdevplatform/shell/kross/krossdistributedversioncontrol.cpp 942782 
>   /trunk/KDE/kdevplatform/vcs/dvcs/dvcsplugin.h 942782 
>   /trunk/KDE/kdevplatform/vcs/dvcs/dvcsplugin.cpp 942782 
>   /trunk/KDE/kdevplatform/vcs/interfaces/ibasicversioncontrol.h 942782 
>   /trunk/KDE/kdevplatform/vcs/widgets/vcseventwidget.cpp 942782 
> 
> Diff: http://reviewboard.kde.org/r/395/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Fabian
> 
>





More information about the KDevelop-devel mailing list