mirror of
https://chromium.googlesource.com/crosvm/crosvm
synced 2024-11-29 03:00:19 +00:00
40af8ee748
Enable cargo clippy in the smoke test script for kokoro with many whitelisted rules to be resolved. This check will reject compile warnings as well. Also, this CL removes cargo check in Docker.crosvm as cargo clippy is a superset of cargo check. BUG=chromium:908640 TEST=./wrapped_smoke_test.sh Change-Id: Iaa546fc0166e3d9762c5e91144d036725d38b0f9 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/crosvm/+/2145538 Commit-Queue: Keiichi Watanabe <keiichiw@chromium.org> Tested-by: Keiichi Watanabe <keiichiw@chromium.org> Reviewed-by: Daniel Verkamp <dverkamp@chromium.org> Reviewed-by: Zach Reizner <zachr@chromium.org>
76 lines
1.8 KiB
Bash
Executable file
76 lines
1.8 KiB
Bash
Executable file
#!/bin/bash
|
|
|
|
# Copyright 2019 The Chromium OS Authors. All rights reserved.
|
|
# Use of this source code is governed by a BSD-style license that can be
|
|
# found in the LICENSE file.
|
|
|
|
# Run `cargo clippy` on all Rust code in crosvm with a mindful set of lints
|
|
# suppressed.
|
|
|
|
set -euo pipefail
|
|
|
|
# Change into directory of script, which is crosvm/bin.
|
|
cd "$(dirname "${BASH_SOURCE[0]}")"
|
|
|
|
# Jump up to root directory of crosvm repo.
|
|
cd ..
|
|
|
|
SUPPRESS=(
|
|
# TODO(crbug/908640): To be resolved.
|
|
borrowed_box
|
|
char_lit_as_u8
|
|
clone_on_copy
|
|
collapsible_if
|
|
comparison_chain
|
|
extra_unused_lifetimes
|
|
for_kv_map
|
|
inefficient_to_string
|
|
into_iter_on_ref
|
|
let_unit_value
|
|
missing_safety_doc
|
|
needless_doctest_main
|
|
needless_range_loop
|
|
needless_return
|
|
option_map_unit_fn
|
|
question_mark
|
|
range_plus_one
|
|
redundant_clone
|
|
redundant_closure
|
|
single_match
|
|
slow_vector_initialization
|
|
unnecessary_filter_map
|
|
unnecessary_mut_passed
|
|
unneeded_field_pattern
|
|
useless_format
|
|
wrong_self_convention
|
|
|
|
# We don't care about these lints. Okay to remain suppressed globally.
|
|
blacklisted_name
|
|
cast_lossless
|
|
cognitive_complexity
|
|
enum_variant_names
|
|
identity_op
|
|
len_without_is_empty
|
|
len_zero
|
|
match_bool
|
|
match_wild_err_arm
|
|
module_inception
|
|
needless_bool
|
|
new_without_default
|
|
or_fun_call
|
|
should_implement_trait
|
|
single_char_pattern
|
|
too_many_arguments
|
|
transmute_ptr_to_ptr
|
|
trivially_copy_pass_by_ref
|
|
type_complexity
|
|
unreadable_literal
|
|
useless_let_if_seq
|
|
useless_transmute
|
|
)
|
|
|
|
# Needed or else clippy won't re-run on code that has already compiled.
|
|
cargo clean
|
|
|
|
cargo clippy --all-features --all-targets -- ${SUPPRESS[@]/#/-Aclippy::} "$@" \
|
|
-D warnings
|