kdesvn: cleanup/c++11'ify
ClosedPublic

Authored by chehrlic on Jul 31 2019, 10:00 AM.

Details

Reviewers
dfaure
Summary

These were 11 separate patches in git but arc merged them into one. For review this should be fine, will commit the patches separately later on.

Diff Detail

Repository
R220 Subversion Client (KDE SVN)
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 14599
Build 14617: arc lint + arc unit
chehrlic requested review of this revision.Jul 31 2019, 10:00 AM
chehrlic created this revision.
chehrlic updated this revision to Diff 62835.Jul 31 2019, 10:00 AM
  • svnqt/cache: use range-based for loops
  • svnfrontend: use range-based for loops
  • svnfrontend/models: use range-based for loops
  • svnfrontend: use range-based for loops, use member-initialization instead custom ctors, use auto instead foo<bar>::iterator
  • svnfrontend/svnactions: use range-based for loops
dfaure accepted this revision.Aug 1 2019, 7:52 AM

I tried hard to find a problem, but I couldn't. Good job :-)

This revision is now accepted and ready to land.Aug 1 2019, 7:52 AM
chehrlic closed this revision.Aug 1 2019, 9:39 AM

uploaded without arcanist to have 11 separate commits