Dartest CL - Please review

compiler/java/com/google/dart/compiler/CommandLineOptions.java

Index: compiler/java/com/google/dart/compiler/CommandLineOptions.java
===================================================================
--- compiler/java/com/google/dart/compiler/CommandLineOptions.java	(revision 2358)
+++ compiler/java/com/google/dart/compiler/CommandLineOptions.java	(working copy)
@@ -73,13 +73,13 @@
         usage = "Generate source maps")
     private boolean generateSourceMaps = false;
     
-    @Option(name = "--dump-ast-format", aliases={"-dump-ast-format"},
-    		usage = "Dump parse tree. Supported formats include console, text or dot")
+    @Option(name = "--dump_ast_format",
+        usage = "Dump parse tree. Supported formats include console, text or dot")
     private String dumpAST = "";
     
-    @Option(name = "--coverage-instrumenter", aliases={"-coverage-instrumentor"},
-    		usage = "Add coverage instrumentation probes")
-    private boolean coverage = false;
+    @Option(name = "--coverage_type",
+        usage = "Add instrumentation probes for collecting coverage. Supported types include function, statement and branch")

[zundel, 2011/12/16 20:49:05]

line > 100 chars

+    private String coverage = "";
 
     @Option(name = "--human-readable-output",
         usage = "Write human readable javascript")
@@ -256,11 +256,11 @@
     }
     
     public String dumpAST(){
-    	return dumpAST;
+      return dumpAST;
     }
     
-    public boolean instrument4Coverage(){
-    	return coverage;
+    public String getCoverageType(){
+      return coverage;
     }
 
     /**