fixed interger division bug and related tests, as well as a comment bug

This commit is contained in:
Armas Spann
2022-04-07 01:16:06 +02:00
parent 272be2aaad
commit 3cd6eb13a9
2 changed files with 3 additions and 3 deletions

View File

@@ -216,7 +216,7 @@ pub enum AuraZone {
/// Default factory modes structure. This easily converts to an USB HID packet with:
/// ```rust
/// let bytes: [u8; LED_MSG_LEN] = mode.into();
/// // let bytes: [u8; LED_MSG_LEN] = mode.into();
/// ```
#[cfg_attr(feature = "dbus", derive(Type))]
#[derive(Debug, Clone, Deserialize, Serialize)]

View File

@@ -71,10 +71,10 @@ impl std::str::FromStr for CurveData {
} else {
let mut p = r;
if percentages {
p *= 255 / 100;
if r > 100 {
return Err(ProfileError::ParseFanCurvePercentOver100(r));
}
p = (p as f32 * 2.55).round() as u8;
}
if pwm_prev > p {
return Err(ProfileError::ParseFanCurvePrevHigher(
@@ -222,7 +222,7 @@ mod tests {
.unwrap();
assert_eq!(curve.fan, FanCurvePU::CPU);
assert_eq!(curve.temp, [30, 49, 59, 69, 79, 89, 99, 109]);
assert_eq!(curve.pwm, [1, 2, 3, 4, 31, 49, 56, 58]);
assert_eq!(curve.pwm, [3, 5, 8, 10, 79, 125, 143, 148]);
}
#[test]