Skip to content
This repository has been archived by the owner on Aug 31, 2023. It is now read-only.

🐛 Control flow visitor: Index out of bounds #3390

Closed
1 task done
MichaReiser opened this issue Oct 11, 2022 · 1 comment · Fixed by #3400
Closed
1 task done

🐛 Control flow visitor: Index out of bounds #3390

MichaReiser opened this issue Oct 11, 2022 · 1 comment · Fixed by #3400
Labels
A-Linter Area: linter S-Bug: confirmed Status: report has been confirmed as a valid bug
Milestone

Comments

@MichaReiser
Copy link
Contributor

MichaReiser commented Oct 11, 2022

Environment information

main (2ffe2b7)

What happened?

Running Rome's formatter on the vscode repository gives me the following three out of bounds errors:

Rome encountered an unexpected error

This is a bug in Rome, not an error in your code, and we would appreciate it if you could report it to https://github.com/rome/tools/issues/ along with the following information to help us fixing the issue:

Source Location: crates/rome_js_analyze/src/control_flow/visitor.rs:85:1
Message: index out of bounds: the len is 2 but the index is 4

../vscode/src/vs/workbench/contrib/tasks/browser/abstractTaskService.ts internalError/panic  INTERNAL  

  ✖ processing panicked: index out of bounds: the len is 2 but the index is 4
  
  ⚠ This diagnostic was derived from an internal Rome error. Potential bug, please report it if necessary.
  

Rome encountered an unexpected error

This is a bug in Rome, not an error in your code, and we would appreciate it if you could report it to https://github.com/rome/tools/issues/ along with the following information to help us fixing the issue:

Source Location: crates/rome_js_analyze/src/control_flow/visitor.rs:85:1
Message: index out of bounds: the len is 2 but the index is 2

../vscode/src/vs/platform/terminal/common/capabilities/commandDetectionCapability.ts internalError/panic  INTERNAL  

  ✖ processing panicked: index out of bounds: the len is 2 but the index is 2
  
  ⚠ This diagnostic was derived from an internal Rome error. Potential bug, please report it if necessary.
  

Rome encountered an unexpected error

This is a bug in Rome, not an error in your code, and we would appreciate it if you could report it to https://github.com/rome/tools/issues/ along with the following information to help us fixing the issue:

Source Location: crates/rome_js_analyze/src/control_flow/visitor.rs:85:1
Message: index out of bounds: the len is 3 but the index is 3

../vscode/src/vs/workbench/contrib/debug/browser/debugEditorContribution.ts internalError/panic  INTERNAL  

  ✖ processing panicked: index out of bounds: the len is 3 but the index is 3
  
  ⚠ This diagnostic was derived from an internal Rome error. Potential bug, please report it if necessary.

Expected result

No out-of-bound errors (Or the CFG not running when formatting?)

Code of Conduct

  • I agree to follow Rome's Code of Conduct
@MichaReiser MichaReiser added S-To triage Status: user report of a possible bug that needs to be triaged S-Bug: confirmed Status: report has been confirmed as a valid bug A-Linter Area: linter and removed S-To triage Status: user report of a possible bug that needs to be triaged labels Oct 11, 2022
@MichaReiser
Copy link
Contributor Author

CC: @leops

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A-Linter Area: linter S-Bug: confirmed Status: report has been confirmed as a valid bug
Projects
Status: No status
Development

Successfully merging a pull request may close this issue.

2 participants