refactor(fe): if(x != y) {switch(x)} -> switch(x)

Author: strager

src/quick-lint-js/fe/parse-statement.cpp

@@ -3519,21 +3519,25 @@ void parser::parse_and_visit_import(parse_visitor_base &v) {
     return;
   }
 
-  if (this->peek().type != token_type::string) {
-    switch (this->peek().type) {
-    QLJS_CASE_KEYWORD:
-    case token_type::identifier:
-      this->diag_reporter_->report(diag_cannot_import_from_unquoted_module{
-          .import_name = this->peek().identifier_name(),
-      });
-      break;
+  switch (this->peek().type) {
+  // import fs from 'fs';
+  case token_type::string:
+    this->skip();
+    break;
 
-    default:
-      QLJS_PARSER_UNIMPLEMENTED();
-      break;
-    }
+  // import foo from bar;  // Invalid.
+  QLJS_CASE_KEYWORD:
+  case token_type::identifier:
+    this->diag_reporter_->report(diag_cannot_import_from_unquoted_module{
+        .import_name = this->peek().identifier_name(),
+    });
+    this->skip();
+    break;
+
+  default:
+    QLJS_PARSER_UNIMPLEMENTED();
+    break;
   }
-  this->skip();
 
   this->consume_semicolon_after_statement();
 }