feat(rome_js_parser): EcmaScript @decorators #4252

rome · May 8, 2023 · 21dbb08 · 21dbb08
1 parent fa3e63f
commit 21dbb08
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/crates/rome_js_formatter/src/js/expressions/class_expression.rs b/crates/rome_js_formatter/src/js/expressions/class_expression.rs
@@ -12,7 +12,9 @@ pub(crate) struct FormatJsClassExpression;
 
 impl FormatNodeRule<JsClassExpression> for FormatJsClassExpression {
     fn fmt_fields(&self, node: &JsClassExpression, f: &mut JsFormatter) -> FormatResult<()> {
-        if node.decorators().len() > 0 {
+        if node.decorators().is_empty() {
+            FormatClass::from(&node.clone().into()).fmt(f)
+        } else {
             write!(
                 f,
                 [
@@ -23,13 +25,11 @@ impl FormatNodeRule<JsClassExpression> for FormatJsClassExpression {
                     soft_line_break_or_space()
                 ]
             )
-        } else {
-            FormatClass::from(&node.clone().into()).fmt(f)
         }
     }
 
     fn needs_parentheses(&self, item: &JsClassExpression) -> bool {
-        item.decorators().len() > 0 || item.needs_parentheses()
+        !item.decorators().is_empty() || item.needs_parentheses()
     }
 
     fn fmt_dangling_comments(

diff --git a/crates/rome_js_formatter/src/js/lists/decorator_list.rs b/crates/rome_js_formatter/src/js/lists/decorator_list.rs
@@ -75,7 +75,7 @@ impl FormatRule<JsDecoratorList> for FormatJsDecoratorList {
             let is_export = export.is_some();
 
             let has_decorators_before_export =
-                export.map_or(false, |export| export.decorators().len() > 0);
+                export.map_or(false, |export| !export.decorators().is_empty());
 
             if has_decorators_before_export {
                 return Ok(());

diff --git a/crates/rome_js_formatter/src/utils/assignment_like.rs b/crates/rome_js_formatter/src/utils/assignment_like.rs
@@ -941,7 +941,7 @@ pub(crate) fn should_break_after_operator(
                 })
         }
 
-        AnyJsExpression::JsClassExpression(class) => class.decorators().len() > 0,
+        AnyJsExpression::JsClassExpression(class) => !class.decorators().is_empty(),
 
         _ => false,
     };