about summary refs log tree commit diff
path: root/compiler/rustc_llvm/llvm-wrapper/Linker.cpp
diff options
context:
space:
mode:
authorJacob Pratt <jacob@jhpratt.dev>2024-07-04 04:09:50 -0400
committerGitHub <noreply@github.com>2024-07-04 04:09:50 -0400
commit726b9b7c272debc4426a82b42b0fd594f82c4459 (patch)
tree0e0273133bd9a137dc7c172964fd4c0bf5624b96 /compiler/rustc_llvm/llvm-wrapper/Linker.cpp
parent5712539a627ef2f8bf3ce8889940b3ba4dc55e6f (diff)
parentccc8baf08aa5a204a7df55e67022aa6c5254966c (diff)
downloadrust-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/Linker.cpp')
0 files changed, 0 insertions, 0 deletions