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

Issue 6368032: cros_workon: apply gspencer's cleanup from issue 6240018 (Closed)

Created:
9 years, 10 months ago by Mandeep Singh Baines
Modified:
9 years, 7 months ago
CC:
chromium-os-reviews_chromium.org
Visibility:
Public.

Description

cros_workon: apply gspencer's cleanup from issue 6240018 Note: this is the moved version of cros_workon BUG=11507 TEST=./cros_workon --board x86-generic list --all Change-Id: I4bee245743b4390e1efb634887ea8858c4540f34 Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=f75ed18

Patch Set 1 #

Total comments: 1

Patch Set 2 : Fixed per review. #

Patch Set 3 : more fixup #

Unified diffs Side-by-side diffs Delta from patch set Stats (+20 lines, -13 lines) Patch
M host/cros_workon View 1 2 13 chunks +20 lines, -13 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
Mandeep Singh Baines
9 years, 10 months ago (2011-02-01 21:32:52 UTC) #1
robotboy
Looking for clarification, I may have missed something. Thanks, Anton http://codereview.chromium.org/6368032/diff/1/host/cros_workon File host/cros_workon (right): http://codereview.chromium.org/6368032/diff/1/host/cros_workon#newcode18 ...
9 years, 10 months ago (2011-02-01 21:46:13 UTC) #2
Mandeep Singh Baines
Good point. Removed boilerplate. On 2011/02/01 21:46:13, robotboy wrote: > Looking for clarification, I may ...
9 years, 10 months ago (2011-02-01 21:58:01 UTC) #3
robotboy
LGTM, thanks.
9 years, 10 months ago (2011-02-01 22:00:36 UTC) #4
Greg Spencer (Chromium)
LGTM. You just like seeing me merge things, right? :) (I already applied these changes ...
9 years, 10 months ago (2011-02-01 23:19:49 UTC) #5
Greg Spencer (Chromium)
9 years, 10 months ago (2011-02-01 23:22:08 UTC) #6
On 2011/02/01 23:19:49, Greg Spencer (Chromium) wrote:
> LGTM.  You just like seeing me merge things, right? :)
> 
> (I already applied these changes in my tree, and they'll go in as soon as I've
> tested them on a clean tree).

But do feel free checking this in, I'm just pulling your leg about the merging.

Powered by Google App Engine
This is Rietveld 408576698