fix to prevent IndexOutOfRangeException

This prevents the IndexOutOfRangeException exception when Unity returns NaN for the refresh rate. It also displays the refresh rate with up to 3 decimal places so that rates like 29.97, 59.94, and 144.003 will show up correctly and not truncated.

Author: paulsinnett

Runtime/Advanced/G_AdvancedData.cs

@@ -87,6 +87,10 @@ public class G_AdvancedData : MonoBehaviour, IMovable, IModifiableState
             "Hz"
         };
 
+#if UNITY_2022_2_OR_NEWER
+        private SortedList<RefreshRate, string> m_refreshRates = new SortedList<RefreshRate, string>();
+#endif
+
         #endregion
 
         #region Methods -> Unity Callbacks
@@ -96,6 +100,20 @@ private void Awake()
             Init();
         }
 
+        private string RefreshRateToString( Resolution resolution )
+        {
+#if UNITY_2022_2_OR_NEWER
+            if( !m_refreshRates.TryGetValue( resolution.refreshRateRatio, out string refreshRate ) )
+            {
+                refreshRate = resolution.refreshRateRatio.value.ToString( "0.###" );
+                m_refreshRates.Add( resolution.refreshRateRatio, refreshRate );
+            }
+            return refreshRate;
+#else
+            return resolution.refreshRate.ToStringNonAlloc();
+#endif
+        }
+
         private void Update()
         {
             m_deltaTime += Time.unscaledDeltaTime;
@@ -107,13 +125,7 @@ private void Update()
 
                 m_sb.Append( m_windowStrings[ 0 ] ).Append( Screen.width.ToStringNonAlloc() )
                     .Append( m_windowStrings[ 1 ] ).Append( Screen.height.ToStringNonAlloc() )
-                    .Append( m_windowStrings[ 2 ] ).Append(
-#if UNITY_2022_2_OR_NEWER
-                        ((int)Screen.currentResolution.refreshRateRatio.value).ToStringNonAlloc()
-#else
-                        Screen.currentResolution.refreshRate.ToStringNonAlloc()
-#endif
-                        )
+                    .Append( m_windowStrings[ 2 ] ).Append( RefreshRateToString( Screen.currentResolution ) )
                     .Append( m_windowStrings[ 3 ] )
                     .Append( m_windowStrings[ 4 ] ).Append( ((int) Screen.dpi).ToStringNonAlloc() )
                     .Append( m_windowStrings[ 5 ] );
@@ -333,11 +345,7 @@ private void Init()
                   + "x"
                   + res.height
                   + "@"
-#if UNITY_2022_2_OR_NEWER
-                  + ((int)Screen.currentResolution.refreshRateRatio.value).ToStringNonAlloc()
-#else
-                  + res.refreshRate
-#endif
+                  + RefreshRateToString( res )
                   + "Hz";
 
             m_operatingSystemText.text

Runtime/Util/G_Intstring.cs

@@ -98,7 +98,7 @@ public static void Dispose()
         /// </returns>
         public static string ToStringNonAlloc( this int value )
         {
-            if( value < 0 && -value <= m_negativeBuffer.Length )
+            if( value < 0 && value >= -m_negativeBuffer.Length )
             {
                 return m_negativeBuffer[ -value - 1 ];
             }