From bffb33297feddc1f24d2f40f7a3a0b6528de9cd3 Mon Sep 17 00:00:00 2001 From: ice_iix Date: Tue, 30 Jun 2020 19:01:17 -0700 Subject: [PATCH] Allow float_cmp since we use it to check if floating point values changed from the last value assigned, not compare two different floating point values; downgrade not_unsafe_ptr_arg_deref to warning --- azure-pipelines.yml | 2 +- src/main.rs | 6 ++++-- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/azure-pipelines.yml b/azure-pipelines.yml index 1889625..0445577 100644 --- a/azure-pipelines.yml +++ b/azure-pipelines.yml @@ -6,7 +6,7 @@ jobs: displayName: "Install build dependencies" condition: eq(variables['Agent.OS'], 'Linux') minrust: 1.44.1 - clippy: -A clippy::not_unsafe_ptr_arg_deref -A clippy::float_cmp + clippy: resources: repositories: diff --git a/src/main.rs b/src/main.rs index 9f9d696..971f3d0 100644 --- a/src/main.rs +++ b/src/main.rs @@ -13,8 +13,10 @@ // limitations under the License. #![recursion_limit = "300"] -#![allow(clippy::too_many_arguments)] -#![allow(clippy::many_single_char_names)] +#![allow(clippy::too_many_arguments)] // match standard gl functions with many arguments +#![allow(clippy::many_single_char_names)] // short variable names provide concise clarity +#![warn(clippy::not_unsafe_ptr_arg_deref)] // TODO: fix unsafe warnings +#![allow(clippy::float_cmp)] // float comparison used to check if changed use log::{error, info, warn}; use std::time::{Duration, Instant};