go.tools/present: correctly test empty argument

Updates golang/go#7613. LGTM=r R=r CC=golang-codereviews https://golang.org/cl/105130046
springning · Jun 12, 2014 · 6251f07 · 6251f07
1 parent 53915e2
commit 6251f07
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/present/code.go b/present/code.go
@@ -239,7 +239,11 @@ func parseArgs(name string, line int, args []string) (res []interface{}, err err
 		case '$':
 			res[i] = "$"
 		case '_':
-			// Do nothing; '_' indicates an intentionally empty parameter.
+			if len(v) == 1 {
+				// Do nothing; "_" indicates an intentionally empty parameter.
+				break
+			}
+			fallthrough
 		default:
 			return nil, fmt.Errorf("%s:%d bad code argument %q", name, line, v)
 		}