From abba7fbb78bc9dc26362038a04e077f39c289e2b Mon Sep 17 00:00:00 2001
From: YoshiRulz <OSSYoshiRulz@gmail.com>
Date: Thu, 23 Jan 2020 15:53:19 +1000
Subject: [PATCH] Cleanup Ranges and SimpleTime

---
 BizHawk.Common/Ranges.cs     | 14 ++++++--------
 BizHawk.Common/SimpleTime.cs | 32 ++++++++++++++------------------
 2 files changed, 20 insertions(+), 26 deletions(-)

diff --git a/BizHawk.Common/Ranges.cs b/BizHawk.Common/Ranges.cs
index 3527a59442..30523d85b8 100644
--- a/BizHawk.Common/Ranges.cs
+++ b/BizHawk.Common/Ranges.cs
@@ -1,13 +1,11 @@
-#nullable disable
-
-using System;
-using System.Collections.Generic;
-using System.Linq;
-
-using BizHawk.Common.NumberExtensions;
-
 namespace BizHawk.Common
 {
+	using System;
+	using System.Collections.Generic;
+	using System.Linq;
+
+	using BizHawk.Common.NumberExtensions;
+
 	/// <summary>semantically similar to <see cref="Range{T}"/>, but obviously does no checks at runtime</summary>
 	public struct RangeStruct<T> where T : unmanaged, IComparable<T>
 	{
diff --git a/BizHawk.Common/SimpleTime.cs b/BizHawk.Common/SimpleTime.cs
index d86e3f33b0..745f7d617d 100644
--- a/BizHawk.Common/SimpleTime.cs
+++ b/BizHawk.Common/SimpleTime.cs
@@ -1,31 +1,27 @@
-#nullable disable
-
-using System;
-using System.Diagnostics;
-
-namespace BizHawk.Common
+namespace BizHawk.Common
 {
+	using System;
+	using System.Diagnostics;
+
+	/// <summary>Create a new instance of this class in a <see langword="using"/> block, and it will measure the time elapsed until the block finishes executing. Provide a label to print to stdout or provide a callback for custom behaviour.</summary>
 	public class SimpleTime : IDisposable
 	{
-		private readonly Stopwatch _w;
-		private readonly Action<int> _f;
+		private readonly Action<long> _callback;
 
-		public SimpleTime(string s)
-			: this(t => Console.WriteLine("Elapsed time for {0}: {1}ms", s, t))
+		private readonly Stopwatch _stopwatch = new Stopwatch();
+
+		public SimpleTime(Action<long> callback)
 		{
+			_callback = callback;
+			_stopwatch.Start();
 		}
 
-		public SimpleTime(Action<int> f)
-		{
-			_f = f;
-			_w = new Stopwatch();
-			_w.Start();
-		}
+		public SimpleTime(string label) : this(l => Console.WriteLine($"Elapsed time for {label}: {l} ms")) {}
 
 		public void Dispose()
 		{
-			_w.Stop();
-			_f((int)_w.ElapsedMilliseconds);
+			_stopwatch.Stop();
+			_callback(_stopwatch.ElapsedMilliseconds);
 		}
 	}
 }