diff --git a/.travis.yml b/.travis.yml index 7be917484..6ab711b72 100644 --- a/.travis.yml +++ b/.travis.yml @@ -90,6 +90,7 @@ matrix: allow_failures: - os: windows env: CARGO_INCREMENTAL=0 OS_WINDOWS=true + - env: INTEGRATION=rust-lang-nursery/stdsimd before_script: - | diff --git a/ci/integration-tests.sh b/ci/integration-tests.sh index 6d33224a5..f6540769b 100755 --- a/ci/integration-tests.sh +++ b/ci/integration-tests.sh @@ -18,6 +18,8 @@ cd checkout # run clippy on a project, try to be verbose and trigger as many warnings # as possible for greater coverage +# NOTE: we use `tee` to print any warnings and errors to stdout +# to avoid build timeout in Travis RUST_BACKTRACE=full \ cargo clippy \ --all-targets \ @@ -26,10 +28,9 @@ cargo clippy \ --cap-lints warn \ -W clippy::pedantic \ -W clippy::nursery \ - > clippy_output 2>&1 || true + 2>&1 | tee clippy_output cargo uninstall clippy -cat clippy_output if grep -q "internal compiler error\|query stack during panic\|E0463" clippy_output; then exit 1 diff --git a/clippy_lints/src/mutex_atomic.rs b/clippy_lints/src/mutex_atomic.rs index d4194b0ca..a8c953b88 100644 --- a/clippy_lints/src/mutex_atomic.rs +++ b/clippy_lints/src/mutex_atomic.rs @@ -77,7 +77,7 @@ impl<'a, 'tcx> LateLintPass<'a, 'tcx> for Mutex { } } -fn get_atomic_name(ty: Ty<'_>) -> Option<(&'static str)> { +fn get_atomic_name(ty: Ty<'_>) -> Option<&'static str> { match ty.kind { ty::Bool => Some("AtomicBool"), ty::Uint(_) => Some("AtomicUsize"), diff --git a/clippy_lints/src/utils/mod.rs b/clippy_lints/src/utils/mod.rs index 5729c8c29..bf46a9a8b 100644 --- a/clippy_lints/src/utils/mod.rs +++ b/clippy_lints/src/utils/mod.rs @@ -239,7 +239,7 @@ pub fn match_path_ast(path: &ast::Path, segments: &[&str]) -> bool { } /// Gets the definition associated to a path. -pub fn path_to_res(cx: &LateContext<'_, '_>, path: &[&str]) -> Option<(def::Res)> { +pub fn path_to_res(cx: &LateContext<'_, '_>, path: &[&str]) -> Option { let crates = cx.tcx.crates(); let krate = crates .iter()