refactor(transformer/arrow-function): generate_super_binding_name take &str and &TraverseCtx

[overlookmotel]

Pure refactor. Take &str rather than &Atom (1 less indirection) and &TraverseCtx<'a> (more conventional).

[graphite-app]

Your org has enabled the Graphite merge queue for merging into main

Add the label “0-merge” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix.

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

[overlookmotel]

• perf(transformer/arrow-function): reduce string comparisons #7318
• test(transformer/async-to-generator): failing test for nested supers #7314
• perf(transformer/arrow-functions): move arguments transform checks to aid inlining #7322
• perf(transformer/arrow-functions): store state of whether arguments needs transform #7321
• refactor(transformer/arrow-function): comments on possible improvement #7320
• refactor(transformer/arrow-function): reserve correct capacity for Vec #7319
• perf(transformer/arrow-function): create super method binding names lazily #7313
• refactor(transformer/arrow-functions): use IndexMap for super getter/setters #7317
• perf(transformer/arrow-function): optimize generate_super_binding_name #7312
• fix(transformer/arrow-function): handle unicode when capitalizing property name #7311
• refactor(transformer/arrow-function): generate_super_binding_name take &str and &TraverseCtx #7310 👈 (View in Graphite)
• fix(transformer/async-to-generator): arguments isn't correct after transformation #7234
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[codspeed-hq]

CodSpeed Performance Report

Merging #7310 will not alter performance

_{Comparing 11-16-refactor_transformer_arrow-function_generate_super_binding_name_take_str_and_traversectx_ (5cfe0b6) with main (7d75130)}

Summary

✅ 30 untouched benchmarks

[Dunqing]

Merge activity

• Nov 17, 12:01 AM EST: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
• Nov 17, 12:01 AM EST: A user added this pull request to the Graphite merge queue.
• Nov 17, 12:15 AM EST: The Graphite merge queue couldn't merge this PR because it was not satisfying all requirements (Failed CI: 'Test (ubuntu-latest)', 'Test (macos-14)', 'Clippy', 'Doc', 'Conformance', 'Test NAPI', 'Benchmark (lexer)', 'Benchmark (parser)', 'Benchmark (transformer)', 'Benchmark (isolated_declarations)', 'Benchmark (semantic)', 'Benchmark (minifier)', 'Benchmark (codegen)', 'Benchmark (sourcemap)', 'Check wasm32-unknown-unknown').
• Nov 17, 5:16 AM EST: A user merged this pull request with the Graphite merge queue.

[Dunqing]

I don't know why this conflicted

[overlookmotel]

I don't know why this conflicted

Graphite mangled my PR! It seems to get confused sometimes when downstack PR has multiple commits.