From 800ff53e978207309565a7671eb9f692021910d3 Mon Sep 17 00:00:00 2001 From: rzk3 Date: Thu, 16 Feb 2023 21:12:22 -0500 Subject: [PATCH] Use Math.Clamp instead of if statements --- MediaBrowser.Model/Session/PlaybackProgressInfo.cs | 10 +++------- MediaBrowser.Model/Session/PlayerStateInfo.cs | 10 +++------- 2 files changed, 6 insertions(+), 14 deletions(-) diff --git a/MediaBrowser.Model/Session/PlaybackProgressInfo.cs b/MediaBrowser.Model/Session/PlaybackProgressInfo.cs index c297e45a30..0ea38a5aa5 100644 --- a/MediaBrowser.Model/Session/PlaybackProgressInfo.cs +++ b/MediaBrowser.Model/Session/PlaybackProgressInfo.cs @@ -126,17 +126,13 @@ namespace MediaBrowser.Model.Session set { - if (value > 10) + if (value is null) { - _playbackSpeed = 10; - } - else if (value < 0.1) - { - _playbackSpeed = 0.1; + _playbackSpeed = null; } else { - _playbackSpeed = value; + _playbackSpeed = Math.Clamp((double)value, 0.1, 10.0); } } } diff --git a/MediaBrowser.Model/Session/PlayerStateInfo.cs b/MediaBrowser.Model/Session/PlayerStateInfo.cs index 39126a1f79..d1f0915550 100644 --- a/MediaBrowser.Model/Session/PlayerStateInfo.cs +++ b/MediaBrowser.Model/Session/PlayerStateInfo.cs @@ -88,17 +88,13 @@ namespace MediaBrowser.Model.Session set { - if (value > 10) + if (value is null) { - _playbackSpeed = 10; - } - else if (value < 0.1) - { - _playbackSpeed = 0.1; + _playbackSpeed = null; } else { - _playbackSpeed = value; + _playbackSpeed = Math.Clamp((double)value, 0.1, 10.0); } } }