Review Request: Simplified IBasicVersionControl::diff
Fabian Wiesel
fabian.wiesel at googlemail.com
Sun Mar 22 17:43:39 UTC 2009
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/395/
-----------------------------------------------------------
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