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"},
+    @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")

[pdr, 2011/12/16 17:00:50]

Note: tabs

[shauvik, 2011/12/16 17:57:06]

Done.

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