dan/oxc
1
0
Fork 0
mirror of https://github.com/danbulant/oxc synced 2026-05-24 12:21:58 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

refactor(transformer): simplify match in JSX transform (#5859)

Browse source 
Pure refactor. Reduce match to 1 arm per variant.
This commit is contained in:
overlookmotel 2024-09-19 06:19:10 +00:00
parent a111bb69af
commit 58a83274fd
1 changed files with 11 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
crates/oxc_transformer/src/react/jsx.rs
View file

@ -464,9 +464,8 @@ impl<'a> ReactJsx<'a> {
for attribute in attributes { for attribute in attributes {
match attribute { match attribute {
// optimize `{...prop}` to `prop` in static mode // optimize `{...prop}` to `prop` in static mode
JSXAttributeItem::SpreadAttribute(spread) JSXAttributeItem::SpreadAttribute(spread) => {
if is_classic && attributes.len() == 1 => if is_classic && attributes.len() == 1 {
{
// deopt if spreading an object with `__proto__` key // deopt if spreading an object with `__proto__` key
if !matches!(&spread.argument, Expression::ObjectExpression(o) if o.has_proto()) if !matches!(&spread.argument, Expression::ObjectExpression(o) if o.has_proto())
{ {
@ -477,6 +476,7 @@ impl<'a> ReactJsx<'a> {
continue; continue;
} }
} }
}
JSXAttributeItem::Attribute(attr) => { JSXAttributeItem::Attribute(attr) => {
if attr.is_identifier("__self") { if attr.is_identifier("__self") {
self_attr_span = Some(attr.name.span()); self_attr_span = Some(attr.name.span());
@ -496,7 +496,6 @@ impl<'a> ReactJsx<'a> {
} }
} }
} }
JSXAttributeItem::SpreadAttribute(_) => {}
} }
// Add attribute to prop object // Add attribute to prop object