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

perf(linter/jsx-no-comment-textnodes): remove regex for checking comment patterns (#6534)

Browse source 
Improves performance of this rule by replacing `Regex` with simpler string methods. On the `RadixUIAdoptionSection.jsx` file alone, this improves performance by ~5%:

```
hyperfine --warmup 100 --shell=none --runs 3000 './oxlint-main -W jsx-no-comment-textnodes --silent ./RadixUIAdoptionSection.jsx' './oxlint-jsx-no-comment-textnodes-no-regex -W jsx-no-comment-textnodes --silent ./RadixUIAdoptionSection.jsx'
Benchmark 1: ./oxlint-main -W jsx-no-comment-textnodes --silent ./RadixUIAdoptionSection.jsx
  Time (mean ± σ):       4.4 ms ±   0.1 ms    [User: 1.7 ms, System: 1.9 ms]
  Range (min … max):     4.1 ms …   5.3 ms    3000 runs

Benchmark 2: ./oxlint-jsx-no-comment-textnodes-no-regex -W jsx-no-comment-textnodes --silent ./RadixUIAdoptionSection.jsx
  Time (mean ± σ):       4.2 ms ±   0.1 ms    [User: 1.5 ms, System: 1.8 ms]
  Range (min … max):     3.9 ms …   5.0 ms    3000 runs

Summary
  ./oxlint-jsx-no-comment-textnodes-no-regex -W jsx-no-comment-textnodes --silent ./RadixUIAdoptionSection.jsx ran
    1.06 ± 0.05 times faster than ./oxlint-main -W jsx-no-comment-textnodes --silent ./RadixUIAdoptionSection.jsx
```
This commit is contained in:
camchenry 2024-10-15 05:02:54 +00:00
parent 8d9a2da9f3
commit 62afaa99dd
1 changed files with 7 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
crates/oxc_linter/src/rules/react/jsx_no_comment_textnodes.rs
View file

@ -1,9 +1,7 @@
use lazy_static::lazy_static;
use oxc_ast::AstKind; use oxc_ast::AstKind;
use oxc_diagnostics::OxcDiagnostic; use oxc_diagnostics::OxcDiagnostic;
use oxc_macros::declare_oxc_lint; use oxc_macros::declare_oxc_lint;
use oxc_span::Span; use oxc_span::Span;
use regex::Regex;
use crate::{ use crate::{
context::{ContextHost, LintContext}, context::{ContextHost, LintContext},
@ -60,7 +58,7 @@ impl Rule for JsxNoCommentTextnodes {
return; return;
}; };
if control_patterns(&jsx_text.value) { if has_comment_pattern(&jsx_text.value) {
ctx.diagnostic(jsx_no_comment_textnodes_diagnostic(jsx_text.span)); ctx.diagnostic(jsx_no_comment_textnodes_diagnostic(jsx_text.span));
} }
} }
@ -70,11 +68,12 @@ impl Rule for JsxNoCommentTextnodes {
} }
} }
fn control_patterns(pattern: &str) -> bool { /// Returns true if the given text contains a comment pattern such as `//` or `/*`.
lazy_static! { fn has_comment_pattern(text: &str) -> bool {
static ref CTL_PAT: Regex = Regex::new(r"(?m)^\s*/(/|\*)",).unwrap(); text.lines().any(|line| {
} let line = line.trim();
CTL_PAT.is_match(pattern) line.starts_with("//") || line.starts_with("/*")
})
} }
#[test] #[test]