refactor(oxc_transformer): use enum to represent Pragma

[IWANABETHATGUY]

No description provided.

[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.

[IWANABETHATGUY]

• refactor(oxc_transformer): use enum to represent Pragma #7613 👈 (View in Graphite)
• main

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

[codspeed-hq]

CodSpeed Performance Report

Merging #7613 will not alter performance

_{Comparing 12-03-refactor_use_enum_in_pragma (8af7752) with main (b4f3812)}

Summary

✅ 29 untouched benchmarks

[overlookmotel]

@IWANABETHATGUY Oh I'm sorry. I didn't realize you were working on this.

[overlookmotel]

One question, if you don't mind me asking: Same as in #7553, here you Box the result of the various match arms. What's the advantage of doing that?

[IWANABETHATGUY]

One question, if you don't mind me asking: Same as in #7553, here you Box the result of the various match arms. What's the advantage of doing that?

No special reason, I saw your code and I think it is better than me, I just don't realize early return could avoid Boxing a iter trait.

[overlookmotel]

Ah ha now I understand the Box. Thanks for explaining.