Rewrite some range comparisons for readability

Author: orottier

src/analysis.rs

@@ -33,6 +33,7 @@ const MAX_FFT_SIZE: usize = 32768;
 
 // [spec] This MUST be a power of two in the range 32 to 32768, otherwise an
 // IndexSizeError exception MUST be thrown.
+#[allow(clippy::manual_range_contains)]
 fn assert_valid_fft_size(fft_size: usize) {
     assert!(
         fft_size.is_power_of_two(),
@@ -41,7 +42,7 @@ fn assert_valid_fft_size(fft_size: usize) {
     );
 
     assert!(
-        (MIN_FFT_SIZE..=MAX_FFT_SIZE).contains(&fft_size),
+        fft_size >= MIN_FFT_SIZE && fft_size <= MAX_FFT_SIZE,
         "IndexSizeError - Invalid fft size: {:?} is outside range [{:?}, {:?}]",
         fft_size,
         MIN_FFT_SIZE,
@@ -51,9 +52,10 @@ fn assert_valid_fft_size(fft_size: usize) {
 
 // [spec] If the value of this attribute is set to a value less than 0 or more
 // than 1, an IndexSizeError exception MUST be thrown.
+#[allow(clippy::manual_range_contains)]
 fn assert_valid_smoothing_time_constant(smoothing_time_constant: f64) {
     assert!(
-        (0. ..=1.).contains(&smoothing_time_constant),
+        smoothing_time_constant >= 0. && smoothing_time_constant <= 1.,
         "IndexSizeError - Invalid smoothing time constant: {:?} is outside range [0, 1]",
         smoothing_time_constant
     );

src/spatial.rs

@@ -256,8 +256,8 @@ pub fn azimuth_and_elevation(
     }
 
     // Make azimuth relative to "forward" and not "right" listener vector.
-    let max270 = std::ops::RangeInclusive::new(0., 270.);
-    if max270.contains(&azimuth) {
+    #[allow(clippy::manual_range_contains)]
+    if azimuth >= 0. && azimuth <= 270. {
         azimuth = 90. - azimuth;
     } else {
         azimuth = 450. - azimuth;