Don't allow -noEmit with -out or -outDir

Arnavion · Arnavion · commit df6aa85d4f2a · 2014-12-16T20:25:20.000-08:00
diff --git a/src/compiler/diagnosticInformationMap.generated.ts b/src/compiler/diagnosticInformationMap.generated.ts
@@ -378,6 +378,7 @@ module ts {
         Could_not_write_file_0_Colon_1: { code: 5033, category: DiagnosticCategory.Error, key: "Could not write file '{0}': {1}" },
         Option_mapRoot_cannot_be_specified_without_specifying_sourcemap_option: { code: 5038, category: DiagnosticCategory.Error, key: "Option mapRoot cannot be specified without specifying sourcemap option." },
         Option_sourceRoot_cannot_be_specified_without_specifying_sourcemap_option: { code: 5039, category: DiagnosticCategory.Error, key: "Option sourceRoot cannot be specified without specifying sourcemap option." },
+        Option_noEmit_cannot_be_specified_with_option_out_or_outDir: { code: 5040, category: DiagnosticCategory.Error, key: "Option noEmit cannot be specified with option out or outDir." },
         Concatenate_and_emit_output_to_single_file: { code: 6001, category: DiagnosticCategory.Message, key: "Concatenate and emit output to single file." },
         Generates_corresponding_d_ts_file: { code: 6002, category: DiagnosticCategory.Message, key: "Generates corresponding '.d.ts' file." },
         Specifies_the_location_where_debugger_should_locate_map_files_instead_of_generated_locations: { code: 6003, category: DiagnosticCategory.Message, key: "Specifies the location where debugger should locate map files instead of generated locations." },
diff --git a/src/compiler/diagnosticMessages.json b/src/compiler/diagnosticMessages.json
@@ -1511,6 +1511,10 @@
         "category": "Error",
         "code": 5039
     },
+    "Option noEmit cannot be specified with option out or outDir.": {
+        "category": "Error",
+        "code": 5040
+    },
     "Concatenate and emit output to single file.": {
         "category": "Message",
         "code": 6001
diff --git a/src/compiler/parser.ts b/src/compiler/parser.ts
@@ -5539,6 +5539,12 @@ module ts {
                     commonSourceDirectory += directorySeparator;
                 }
             }
+
+            if (options.noEmit) {
+                if (options.out || options.outDir) {
+                    errors.push(createCompilerDiagnostic(Diagnostics.Option_noEmit_cannot_be_specified_with_option_out_or_outDir));
+                }
+            }
         }
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -5539,6 +5539,12 @@ module ts {`
`5539`	`5539`	`commonSourceDirectory += directorySeparator;`
`5540`	`5540`	`}`
`5541`	`5541`	`}`
	`5542`	`+`
	`5543`	`+ if (options.noEmit) {`
	`5544`	`+ if (options.out \|\| options.outDir) {`
	`5545`	`+ errors.push(createCompilerDiagnostic(Diagnostics.Option_noEmit_cannot_be_specified_with_option_out_or_outDir));`
	`5546`	`+ }`
	`5547`	`+ }`
`5542`	`5548`	`}`
`5543`	`5549`	`}`
`5544`	`5550`	`}`