Use cargo cranky instead of cargo clippy (#1820)

* Use cargo cranky instead of cargo clippy

cargo cranky (https://github.com/ericseppanen/cargo-cranky)
is a new tool that passes lints specified in a Cranky.toml
to cargo clippy.

This is a possible solution to
rust-lang/cargo#5034

* Remove `-W clippy::all` from `check.sh` (rely on `Cranky.toml` instead)

.github/workflows/rust.yml

@@ -142,8 +142,8 @@ jobs:
           command: fmt
           args: --all -- --check
 
-  clippy:
-    name: cargo clippy
+  cranky:
+    name: cargo cranky
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
@@ -152,14 +152,14 @@ jobs:
           profile: minimal
           toolchain: 1.60.0
           override: true
-      - run: rustup component add clippy
+      - run: cargo install cargo-cranky
       - name: Install packages (Linux)
         if: runner.os == 'Linux'
         run: sudo apt-get update && sudo apt-get install libspeechd-dev libgtk-3-dev # libgtk-3-dev is used by rfd
       - uses: actions-rs/cargo@v1
         with:
-          command: clippy
-          args: --workspace --all-targets --all-features --  -D warnings -W clippy::all
+          command: cranky
+          args: --workspace --all-targets --all-features --  -D warnings
 
   doc:
     name: cargo doc

Cranky.toml

@@ -0,0 +1,106 @@
+# https://github.com/ericseppanen/cargo-cranky
+# cargo install cargo-cranky && cargo cranky
+
+deny = [
+  "unsafe_code",
+]
+
+warn = [
+    "clippy::all",
+    "clippy::await_holding_lock",
+    "clippy::char_lit_as_u8",
+    "clippy::checked_conversions",
+    "clippy::dbg_macro",
+    "clippy::debug_assert_with_mut_call",
+    "clippy::disallowed_methods",
+    "clippy::disallowed_script_idents",
+    "clippy::doc_markdown",
+    "clippy::empty_enum",
+    "clippy::enum_glob_use",
+    "clippy::equatable_if_let",
+    "clippy::exit",
+    "clippy::expl_impl_clone_on_copy",
+    "clippy::explicit_deref_methods",
+    "clippy::explicit_into_iter_loop",
+    "clippy::fallible_impl_from",
+    "clippy::filter_map_next",
+    "clippy::flat_map_option",
+    "clippy::float_cmp_const",
+    "clippy::fn_params_excessive_bools",
+    "clippy::fn_to_numeric_cast_any",
+    "clippy::from_iter_instead_of_collect",
+    "clippy::if_let_mutex",
+    "clippy::implicit_clone",
+    "clippy::imprecise_flops",
+    "clippy::index_refutable_slice",
+    "clippy::inefficient_to_string",
+    "clippy::invalid_upcast_comparisons",
+    "clippy::iter_not_returning_iterator",
+    "clippy::large_digit_groups",
+    "clippy::large_stack_arrays",
+    "clippy::large_types_passed_by_value",
+    "clippy::let_unit_value",
+    "clippy::linkedlist",
+    "clippy::lossy_float_literal",
+    "clippy::macro_use_imports",
+    "clippy::manual_ok_or",
+    "clippy::map_err_ignore",
+    "clippy::map_flatten",
+    "clippy::map_unwrap_or",
+    "clippy::match_on_vec_items",
+    "clippy::match_same_arms",
+    "clippy::match_wild_err_arm",
+    "clippy::match_wildcard_for_single_variants",
+    "clippy::mem_forget",
+    "clippy::mismatched_target_os",
+    "clippy::missing_enforced_import_renames",
+    "clippy::missing_errors_doc",
+    "clippy::missing_safety_doc",
+    "clippy::mut_mut",
+    "clippy::mutex_integer",
+    "clippy::needless_borrow",
+    "clippy::needless_continue",
+    "clippy::needless_for_each",
+    "clippy::needless_pass_by_value",
+    "clippy::negative_feature_names",
+    "clippy::nonstandard_macro_braces",
+    "clippy::option_option",
+    "clippy::path_buf_push_overwrite",
+    "clippy::ptr_as_ptr",
+    "clippy::rc_mutex",
+    "clippy::ref_option_ref",
+    "clippy::rest_pat_in_fully_bound_structs",
+    "clippy::same_functions_in_if_condition",
+    "clippy::semicolon_if_nothing_returned",
+    "clippy::single_match_else",
+    "clippy::str_to_string",
+    "clippy::string_add_assign",
+    "clippy::string_add",
+    "clippy::string_lit_as_bytes",
+    "clippy::string_to_string",
+    "clippy::todo",
+    "clippy::trailing_empty_array",
+    "clippy::trait_duplication_in_bounds",
+    "clippy::unimplemented",
+    "clippy::unnecessary_wraps",
+    "clippy::unnested_or_patterns",
+    "clippy::unused_self",
+    "clippy::useless_transmute",
+    "clippy::verbose_file_reads",
+    "clippy::zero_sized_map_values",
+    "future_incompatible",
+    "nonstandard_style",
+    "rust_2018_idioms",
+    "rustdoc::missing_crate_level_docs",
+    "semicolon_in_expressions_from_macros",
+    "trivial_numeric_casts",
+    "unused_extern_crates",
+    "unused_import_braces",
+    # "clippy::cloned_instead_of_copied",
+    # "clippy::mod_module_files",
+    # "trivial_casts",
+    # "unused_qualifications",
+]
+
+allow = [
+]

egui-wgpu/src/winit.rs

@@ -179,8 +179,8 @@ impl<'a> Painter<'a> {
                 self.ensure_render_state_for_surface(&surface);
 
                 let size = window.inner_size();
-                let width = size.width as u32;
-                let height = size.height as u32;
+                let width = size.width;
+                let height = size.height;
                 self.surface_state = Some(SurfaceState {
                     surface,
                     width,

sh/check.sh

@@ -6,14 +6,16 @@ set -eux
 # Checks all tests, lints etc.
 # Basically does what the CI does.
 
+cargo install cargo-cranky # Uses lints defined in Cranky.toml. See https://github.com/ericseppanen/cargo-cranky
+
 RUSTFLAGS="-D warnings"
 RUSTDOCFLAGS="-D warnings" # https://github.com/emilk/egui/pull/1454
 
 cargo check --workspace --all-targets
 cargo check --workspace --all-targets --all-features
 cargo check -p egui_demo_app --lib --target wasm32-unknown-unknown
 cargo check -p egui_demo_app --lib --target wasm32-unknown-unknown --all-features
-cargo clippy --workspace --all-targets --all-features --  -D warnings -W clippy::all
+cargo cranky --workspace --all-targets --all-features -- -D warnings
 cargo test --workspace --all-targets --all-features
 cargo test --workspace --doc # slow - checks all doc-tests
 cargo fmt --all -- --check