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

Issue 391075: Revert 32057, 32058, 32059, 32062 because they still have unwanted side-effects. (Closed)

Created:
11 years, 1 month ago by M-A Ruel
Modified:
9 years, 7 months ago
Reviewers:
Dirk Pranke
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

Revert 32057, 32058, 32059, 32062 because they still have unwanted side-effects. Revert "Group SCM-specific functions in classes to simplify generalization of the interface." Revert "Fix 2 recently introduced errors." Revert "Fix a variable aliasing error." Revert "Fix typo." TBR=dpranke TEST=none BUG=none

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+679 lines, -699 lines) Patch
M gcl.py View 12 chunks +44 lines, -14 lines 0 comments Download
M gclient.py View 3 chunks +4 lines, -4 lines 0 comments Download
M gclient_scm.py View 19 chunks +62 lines, -47 lines 0 comments Download
M gclient_utils.py View 7 chunks +11 lines, -11 lines 0 comments Download
M git_cl_hooks.py View 2 chunks +3 lines, -2 lines 0 comments Download
M presubmit_support.py View 13 chunks +17 lines, -17 lines 0 comments Download
M revert.py View 1 chunk +1 line, -1 line 0 comments Download
M scm.py View 1 chunk +322 lines, -402 lines 0 comments Download
M tests/gcl_unittest.py View 4 chunks +27 lines, -20 lines 0 comments Download
M tests/gclient_scm_test.py View 11 chunks +33 lines, -43 lines 0 comments Download
M tests/gclient_test.py View 2 chunks +12 lines, -5 lines 0 comments Download
M tests/presubmit_unittest.py View 21 chunks +59 lines, -65 lines 0 comments Download
M tests/revert_unittest.py View 2 chunks +1 line, -2 lines 0 comments Download
M tests/scm_unittest.py View 11 chunks +33 lines, -57 lines 0 comments Download
M tests/super_mox.py View 1 chunk +1 line, -1 line 0 comments Download
M tests/trychange_unittest.py View 1 chunk +3 lines, -3 lines 0 comments Download
M trychange.py View 3 chunks +46 lines, -5 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
M-A Ruel
11 years, 1 month ago (2009-11-16 18:24:42 UTC) #1
Dirk Pranke
11 years, 1 month ago (2009-11-17 01:35:39 UTC) #2
lgtm after a quick skimming-over

Powered by Google App Engine
This is Rietveld 408576698