diff options
| author | Jacob Pratt <jacob@jhpratt.dev> | 2024-07-04 04:09:50 -0400 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-07-04 04:09:50 -0400 |
| commit | 726b9b7c272debc4426a82b42b0fd594f82c4459 (patch) | |
| tree | 0e0273133bd9a137dc7c172964fd4c0bf5624b96 /compiler/rustc_llvm/llvm-wrapper/RustWrapper.cpp | |
| parent | 5712539a627ef2f8bf3ce8889940b3ba4dc55e6f (diff) | |
| parent | ccc8baf08aa5a204a7df55e67022aa6c5254966c (diff) | |
| download | rust-726b9b7c272debc4426a82b42b0fd594f82c4459.tar.gz rust-726b9b7c272debc4426a82b42b0fd594f82c4459.zip | |
Rollup merge of #127287 - aDotInTheVoid:jsondocck-index, r=GuillaumeGomez
jsondocck: Use correct index for error message. If you misused a count command like ``@count` $some.selector '"T'"`, you would panic with OOB: ``` thread 'main' panicked at src/tools/jsondocck/src/main.rs:76:92: index out of bounds: the len is 2 but the index is 2 ``` This is because 57c85bd97da3 removed the file param, but didn't update the error case. We now error with: ``` Invalid command: Second argument to `@count` must be a valid usize (got `"T"`) on line 20 ``` As some point I want to rewrite this code to avoid indexing in general, but this is a nice small fix. r? `@GuillaumeGomez`
Diffstat (limited to 'compiler/rustc_llvm/llvm-wrapper/RustWrapper.cpp')
0 files changed, 0 insertions, 0 deletions
