Review Request: Simplified IBasicVersionControl::diff

Andreas Pakulat apaku at gmx.de
Mon Mar 23 18:30:04 UTC 2009


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



/trunk/KDE/kdevplatform/vcs/interfaces/ibasicversioncontrol.h
<http://reviewboard.kde.org/r/395/#comment344>

    These namespace changes seem to be unrelated and I don't see the need for them.



/trunk/KDE/kdevplatform/vcs/interfaces/ibasicversioncontrol.h
<http://reviewboard.kde.org/r/395/#comment345>

    Didn't we agree that we should use a plain Url?


- Andreas


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