Address review feedback: simplify modifier handling

Remove unnecessary empty check and continue statement. The check for at_modifier() on line 322 ensures modifiers is never empty when we reach the Modifiers::new() call.

Co-authored-by: camc314 <[email protected]>

Author: Copilot

crates/oxc_parser/src/modifiers.rs

@@ -332,18 +332,12 @@ impl<'a> ParserImpl<'a> {
             let modifier = self.modifier(kind, self.end_span(span));
             if modifier.kind == ModifierKind::Export {
                 self.error(diagnostics::modifier_already_seen(&modifier));
-                // Don't add the duplicate export modifier to the list
-                continue;
             }
             self.check_for_duplicate_modifiers(flags, &modifier);
             flags.set(modifier_flags, true);
             modifiers.push(modifier);
         }
-        if modifiers.is_empty() {
-            Modifiers::empty()
-        } else {
-            Modifiers::new(Some(modifiers), flags)
-        }
+        Modifiers::new(Some(modifiers), flags)
     }
 
     fn at_modifier(&mut self) -> bool {