Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(253)

Issue 3126020: Remove code duplication and improve style. (Closed)

Created:
10 years, 4 months ago by M-A Ruel
Modified:
9 years, 7 months ago
Reviewers:
bradnelson, bradn
CC:
chromium-reviews
Visibility:
Public.

Description

Remove code duplication and improve style. Create a Popen function to reduce code duplication. Use RemoveDirectory where relevant. Make drover slightly more posix friendly. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=56893

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+55 lines, -105 lines) Patch
M drover.py View 7 chunks +19 lines, -53 lines 0 comments Download
M gcl.py View 1 chunk +2 lines, -8 lines 0 comments Download
M gclient_scm.py View 1 chunk +1 line, -1 line 0 comments Download
M gclient_utils.py View 6 chunks +20 lines, -21 lines 0 comments Download
M scm.py View 2 chunks +3 lines, -11 lines 0 comments Download
M tests/gclient_utils_test.py View 1 chunk +1 line, -1 line 0 comments Download
M tests/scm_unittest.py View 3 chunks +9 lines, -10 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
M-A Ruel
10 years, 4 months ago (2010-08-19 20:02:11 UTC) #1
M-A Ruel
ping
10 years, 4 months ago (2010-08-20 17:43:31 UTC) #2
bradn
10 years, 4 months ago (2010-08-20 18:15:55 UTC) #3
LGTM

Powered by Google App Engine
This is Rietveld 408576698