refactor(minifier): clean up try_compress_typeof_undefined#12958
refactor(minifier): clean up try_compress_typeof_undefined#12958graphite-app[bot] merged 1 commit intomainfrom
try_compress_typeof_undefined#12958Conversation
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. |
There was a problem hiding this comment.
Pull Request Overview
This PR refactors the try_compress_typeof_undefined function to improve code clarity and simplify the logic flow. The changes reorganize conditional checks and eliminate unnecessary pattern matching.
Key changes:
- Reorders conditional checks to improve readability
- Simplifies pattern matching and eliminates unreachable code
- Consolidates the final expression creation logic
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| crates/oxc_minifier/src/peephole/substitute_alternate_syntax.rs | Refactors try_compress_typeof_undefined function to improve code flow and eliminate redundant pattern matching |
| crates/oxc_minifier/examples/minifier.rs | Comments out a debug print statement |
| crates/oxc_ast/src/ast_impl/js.rs | Adds inline annotation to is_specific_string_literal method |
CodSpeed Instrumentation Performance ReportMerging #12958 will not alter performanceComparing Summary
|
Merge activity
|
36c11e9 to
e5e2496
Compare
No description provided.