D8588: Make VCS data container classes implicitly shared

[Aleix Pol Gonzalez]

apol accepted this revision.
apol added a comment.
This revision is now accepted and ready to land.


  I agree this is better.
  
  Let's be more liberal with reviews? I just saw 2 reviews whose end was to remove tests.
  
  Let's agree that verbosity in tests isn't the worst in life, we have _very_ bard code in KDevelop, these tests aren't close by far.
  And thanks @kossebau for taking the time to take our codebase steps forward little by little.

REPOSITORY
  R32 KDevelop

BRANCH
  makedataobjectimplicitelyshared

REVISION DETAIL
  https://phabricator.kde.org/D8588

To: kossebau, #kdevelop, apol
Cc: apol, mwolff, brauch, kdevelop-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20171103/c9c375c2/attachment.html>