Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

codegen: unexpected parentheses wrap in CallExpression #4339

Closed
Dunqing opened this issue Jul 18, 2024 · 0 comments · Fixed by #4509
Closed

codegen: unexpected parentheses wrap in CallExpression #4339

Dunqing opened this issue Jul 18, 2024 · 0 comments · Fixed by #4509
Assignees
@Boshen
Labels
A-codegen Area - Code Generation C-bug Category - Bug P-high Priority - High

Comments

@Dunqing
Copy link
Member

Dunqing commented Jul 18, 2024

See: rolldown/rolldown#1656 (comment)
Caused by #4245

In

a().b

Out

(a()).b

Expected

a().b
@Dunqing Dunqing added C-bug Category - Bug A-codegen Area - Code Generation labels Jul 18, 2024
@Boshen Boshen self-assigned this Jul 27, 2024
@Boshen Boshen added the P-high Priority - High label Jul 27, 2024
Boshen added a commit that referenced this issue Jul 28, 2024
@Boshen
feat(codegen): align operator precedence with esbuild
9984b18 
closes #4339
@Boshen Boshen mentioned this issue Jul 28, 2024
Merged
Boshen added a commit that referenced this issue Jul 28, 2024
@Boshen
feat(codegen): align operator precedence with esbuild (#4509)
35654e6 
closes #4339
@Boshen Boshen closed this as completed Jul 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Boshen Boshen

Labels
A-codegen Area - Code Generation C-bug Category - Bug P-high Priority - High
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(codegen): align operator precedence with esbuild oxc-project/oxc
2 participants
@Boshen @Dunqing