From 5412a7489070d5a75967271c69739f606a895c8a Mon Sep 17 00:00:00 2001 From: YoshiRulz Date: Tue, 5 Mar 2019 19:16:20 +1000 Subject: [PATCH] Use the *correct* formatting string, skip calls to Math.Abs --- BizHawk.Client.Common/tools/Watch/ByteWatch.cs | 2 +- BizHawk.Client.Common/tools/Watch/DWordWatch.cs | 2 +- BizHawk.Client.Common/tools/Watch/WordWatch.cs | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/BizHawk.Client.Common/tools/Watch/ByteWatch.cs b/BizHawk.Client.Common/tools/Watch/ByteWatch.cs index e576b01edd..50531248ba 100644 --- a/BizHawk.Client.Common/tools/Watch/ByteWatch.cs +++ b/BizHawk.Client.Common/tools/Watch/ByteWatch.cs @@ -200,7 +200,7 @@ namespace BizHawk.Client.Common /// Get a string representation of difference /// between current value and the previous one /// - public override string Diff => $"{(byte)Math.Abs(_value - _previous):+G;-G;G}"; + public override string Diff => $"{_value - (short)_previous:+#;-#;0}"; /// /// Get the maximum possible value diff --git a/BizHawk.Client.Common/tools/Watch/DWordWatch.cs b/BizHawk.Client.Common/tools/Watch/DWordWatch.cs index 79a9ef03aa..360966dac0 100644 --- a/BizHawk.Client.Common/tools/Watch/DWordWatch.cs +++ b/BizHawk.Client.Common/tools/Watch/DWordWatch.cs @@ -231,7 +231,7 @@ namespace BizHawk.Client.Common /// Get a string representation of difference /// between current value and the previous one /// - public override string Diff => (_previous - _value).ToString(); + public override string Diff => $"{_value - (long)_previous:+#;-#;0}"; /// /// Get the maximum possible value diff --git a/BizHawk.Client.Common/tools/Watch/WordWatch.cs b/BizHawk.Client.Common/tools/Watch/WordWatch.cs index 1fb49b7823..be6d461bf4 100644 --- a/BizHawk.Client.Common/tools/Watch/WordWatch.cs +++ b/BizHawk.Client.Common/tools/Watch/WordWatch.cs @@ -214,7 +214,7 @@ namespace BizHawk.Client.Common /// Get a string representation of difference /// between current value and the previous one /// - public override string Diff => $"{(ushort)Math.Abs(_value - _previous):+G;-G;G}"; + public override string Diff => $"{_value - (int)_previous:+#;-#;0}"; /// /// Get the maximum possible value