commit 32f3c4ade43dd87e75c083a827460b864b9476ba
from: Russ Cox <rsc@swtch.com>
date: Mon Nov 17 02:28:05 2014 UTC

codereview: fix a few bugs

Change-Id: Ifd9fda05e15c9e1e106ffd4e30e1dafe8423cdf4

commit - 95fee1d42e537209727d9493d1ea3784a8af310a
commit + 32f3c4ade43dd87e75c083a827460b864b9476ba
blob - 307091a7318ee6e186925e7eae5af5cfc708fbe7
blob + 6fd3e11a7e22601d455f28cb87b502aec8e5b21c
--- bin/codereview
+++ bin/codereview
@@ -46,23 +46,24 @@ case pending
 
 case create
 	shift
-	if(! ~ $#* 1) {
+	if(~ $#* 0) {
 		echo 'usage: codereview create branchname' >[1=2]
 		exit usage
 	}
 	branch=$1
-	if(! git branch -l | 9 grep '\* master$') {
+	shift
+	if(! git branch -l | 9 grep '\* master$' >/dev/null) {
 		echo 'codereview: create not on master branch; use codereview commit' >[1=2]
 		exit master
 	}
 	if($show) {
 		echo '% git branch '$branch' && git commit || git branch -d '$branch >[1=2]
 	}
-	git branch $branch && git commit $* || git branch -d $branch
+	git branch $branch && git checkout $branch && git commit $* || git branch -d $branch
 
 case commit
 	shift
-	if(git branch -l | 9 grep '\* master$') {
+	if(git branch -l | 9 grep '\* master$' >/dev/null) {
 		echo 'codereview: commit on master branch; use codereview create <branchname>' >[1=2]
 		exit master
 	}
@@ -75,7 +76,7 @@ case commit
 	exit $status
 
 case upload
-	if(git branch -l | 9 grep '\* master$') {
+	if(git branch -l | 9 grep '\* master$' >/dev/null) {
 		echo 'codereview: upload on master branch' >[1=2]
 		exit master
 	}
@@ -111,7 +112,7 @@ case sync
 	}
 
 	# Exactly this commit in master. Fast-forward from master and delete this branch.
-	if(git branch -r --contains HEAD | 9 grep '^ *origin/master$') {
+	if(git branch -r --contains HEAD | 9 grep '^ *origin/master$' >/dev/null) {
 		$git checkout -q master
 		$git merge -q --ff-only origin/master
 		$git branch -q -d $branch
@@ -124,7 +125,7 @@ case sync
 		exit changeid
 	}
 
-	if(git log --grep 'Change-Id: '$changeid origin/master | 9 grep .) {
+	if(git log --grep 'Change-Id: '$changeid origin/master | 9 grep . >/dev/null) {
 		# Something like this got submitted.
 		$git checkout -q master
 		$git merge -q --ff-only origin/master
@@ -133,7 +134,7 @@ case sync
 		exit 1
 	}
 	
-	if(git branch -r --contains master | 9 grep '^ *origin/master$')
+	if(git branch -r --contains master | 9 grep '^ *origin/master$' >/dev/null)
 		$git branch -f master origin/master
 	$git rebase -q origin/master