refactor(linter): use BigIntLiteral::raw field (#7660)

overlookmotel · overlookmotel · commit a0973dcb5af2 · 2024-12-05T04:23:14.000Z
No need to slice source text to get raw value of `BigIntLiteral`. It already has a `raw` field.
diff --git a/crates/oxc_linter/src/rules/unicorn/numeric_separators_style.rs b/crates/oxc_linter/src/rules/unicorn/numeric_separators_style.rs
@@ -101,13 +101,11 @@ impl Rule for NumericSeparatorsStyle {
                 }
             }
             AstKind::BigIntLiteral(number) => {
-                let raw = number.span.source_text(ctx.source_text());
-
-                if self.only_if_contains_separator && !raw.contains('_') {
+                if self.only_if_contains_separator && !number.raw.contains('_') {
                     return;
                 }
 
-                let formatted = self.format_bigint(number, raw);
+                let formatted = self.format_bigint(number, &number.raw);
 
                 if formatted.len() != number.span.size() as usize {
                     ctx.diagnostic_with_fix(

Original file line number	Diff line number	Diff line change
`@@ -101,13 +101,11 @@ impl Rule for NumericSeparatorsStyle {`
`101`	`101`	`}`
`102`	`102`	`}`
`103`	`103`	`AstKind::BigIntLiteral(number) => {`
`104`		`- let raw = number.span.source_text(ctx.source_text());`
`105`		`-`
`106`		`- if self.only_if_contains_separator && !raw.contains('_') {`
	`104`	`+ if self.only_if_contains_separator && !number.raw.contains('_') {`
`107`	`105`	`return;`
`108`	`106`	`}`
`109`	`107`
`110`		`- let formatted = self.format_bigint(number, raw);`
	`108`	`+ let formatted = self.format_bigint(number, &number.raw);`
`111`	`109`
`112`	`110`	`if formatted.len() != number.span.size() as usize {`
`113`	`111`	`ctx.diagnostic_with_fix(`