about summary refs log tree commit diff
path: root/src/rustllvm/RustWrapper.cpp
diff options
context:
space:
mode:
authorkennytm <kennytm@gmail.com>2018-01-18 01:57:14 +0800
committerGitHub <noreply@github.com>2018-01-18 01:57:14 +0800
commit29c2aa7932ff0cd067f4edbc0bf9731e9dac37b4 (patch)
tree7c10d1398cc3ece403f2153fa39d7b85651e44ef /src/rustllvm/RustWrapper.cpp
parent175dd84ed8969fe136a4da1c7e5fe1d9c7693f2d (diff)
parentf7b48778b1472045bd8b7ee6aacdbf1487efe50b (diff)
downloadrust-29c2aa7932ff0cd067f4edbc0bf9731e9dac37b4.tar.gz
rust-29c2aa7932ff0cd067f4edbc0bf9731e9dac37b4.zip
Rollup merge of #47387 - Rantanen:linkchecker-error-msg, r=steveklabnik
Report errors instead of panic!() when linkcheck encounters absolute paths

The RBE contained some absolute links that failed the link check in #46196. Diagnosing these issues was needlessly complicated, thanks to the linkchecker just panicing instead of reporting proper errors.

This PR replaces the panic with a proper `*errors = true` + error message handling.

The linkchecker itself doesn't have any tests so I intentionally didn't touch anything else than the code that previously did the `panic!()`. A small code quality improvement might be made by binding the `Path::new(base).join(url)` into a variable before the for-loop and using this resolved url in both the for loop and the error message.

r? @steveklabnik

(If not for any other reason than having r on the #46196.)
Diffstat (limited to 'src/rustllvm/RustWrapper.cpp')
0 files changed, 0 insertions, 0 deletions