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

Issue 10832380: Don't shortcut if we're trying to get the latest copy. (Closed)

Created:
8 years, 4 months ago by Emily Fortuna
Modified:
6 years, 10 months ago
Reviewers:
vsm
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Don't shortcut if we're trying to get the latest copy. Committed: https://code.google.com/p/dart/source/detail?r=10926

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -4 lines) Patch
M tools/get_archive.py View 1 chunk +4 lines, -4 lines 1 comment Download

Messages

Total messages: 2 (0 generated)
Emily Fortuna
TBR
8 years, 4 months ago (2012-08-17 21:44:23 UTC) #1
vsm
8 years, 4 months ago (2012-08-20 15:24:45 UTC) #2
lgtm

https://chromiumcodereview.appspot.com/10832380/diff/1/tools/get_archive.py
File tools/get_archive.py (right):

https://chromiumcodereview.appspot.com/10832380/diff/1/tools/get_archive.py#n...
tools/get_archive.py:308: if args.revision and args.revision < 4285:
Consider keeping a table of latest revisions keyed by name at the top.  Then you
could hoist these checks once as:

if args.revision and args.revision < LAST_VALID[positional[0]]:
  ...

Powered by Google App Engine
This is Rietveld 408576698