Patches for git plugin related reviews
Sergey Vidyuk
sir.vestnik at gmail.com
Mon Oct 17 03:14:50 UTC 2011
Hello
Patches for reviews
https://git.reviewboard.kde.org/r/102850/
https://git.reviewboard.kde.org/r/102857/
https://git.reviewboard.kde.org/r/102858/
are attached. Patch for 102857 is modified following the last suggestion from
Milian. I've also uploaded modified version into reviewboard. I'm unsure if
it's good idea to have cpp file for implementation of one empty function.
Actually I'm unsure if it's good idea to have non-pure virtual function in an
interface. If needed I can move implementation back to header or into plugins.
Sergey
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Better-error-handling-in-the-BranchManager-dialog.patch
Type: text/x-patch
Size: 2206 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20111017/6747de41/attachment.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-Git-commit-message-format-validation.patch
Type: text/x-patch
Size: 14217 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20111017/6747de41/attachment-0001.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0003-Restore-git-merge-message-for-unfinished-merge.patch
Type: text/x-patch
Size: 1813 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20111017/6747de41/attachment-0002.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20111017/6747de41/attachment.sig>
More information about the KDevelop-devel
mailing list