Use Ord::clamp to simplify the nice value
				
					
				
			This commit is contained in:
		
							parent
							
								
									c864e10400
								
							
						
					
					
						commit
						eee7c8651d
					
				@ -39,12 +39,8 @@ impl NiceValue {
 | 
				
			|||||||
    /// The raw value given beyond the range are automatically adjusted
 | 
					    /// The raw value given beyond the range are automatically adjusted
 | 
				
			||||||
    /// to the nearest boundary value.
 | 
					    /// to the nearest boundary value.
 | 
				
			||||||
    pub fn new(raw: i8) -> Self {
 | 
					    pub fn new(raw: i8) -> Self {
 | 
				
			||||||
        if raw < Self::MIN.value {
 | 
					        Self {
 | 
				
			||||||
            Self::MIN
 | 
					            value: raw.clamp(Self::MIN.value, Self::MAX.value),
 | 
				
			||||||
        } else if raw > Self::MAX.value {
 | 
					 | 
				
			||||||
            Self::MAX
 | 
					 | 
				
			||||||
        } else {
 | 
					 | 
				
			||||||
            Self { value: raw }
 | 
					 | 
				
			||||||
        }
 | 
					        }
 | 
				
			||||||
    }
 | 
					    }
 | 
				
			||||||
 | 
					
 | 
				
			||||||
@ -56,13 +52,7 @@ impl NiceValue {
 | 
				
			|||||||
 | 
					
 | 
				
			||||||
impl From<i32> for NiceValue {
 | 
					impl From<i32> for NiceValue {
 | 
				
			||||||
    fn from(raw: i32) -> Self {
 | 
					    fn from(raw: i32) -> Self {
 | 
				
			||||||
        let adj_raw = if raw > i8::MAX as i32 {
 | 
					        let adj_raw = raw.clamp(i8::MIN as i32, i8::MAX as i32) as i8;
 | 
				
			||||||
            i8::MAX
 | 
					 | 
				
			||||||
        } else if raw < i8::MIN as i32 {
 | 
					 | 
				
			||||||
            i8::MIN
 | 
					 | 
				
			||||||
        } else {
 | 
					 | 
				
			||||||
            raw as i8
 | 
					 | 
				
			||||||
        };
 | 
					 | 
				
			||||||
        Self::new(adj_raw)
 | 
					        Self::new(adj_raw)
 | 
				
			||||||
    }
 | 
					    }
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
 | 
				
			|||||||
		Loading…
	
		Reference in New Issue
	
	Block a user