Move operators to correct file

Author: xPaw

ValveKeyValue/ValveKeyValue/KVObject.cs

@@ -68,6 +68,7 @@ public KVObject()
 
         /// <summary>
         /// Initializes a new instance of the <see cref="KVObject"/> class with a string value.
+        /// If <paramref name="value"/> is <c>null</c>, creates a null-valued instance.
         /// </summary>
         public KVObject(string value)
         {
@@ -81,56 +82,72 @@ public KVObject(string value)
             _ref = value;
         }
 
-        /// <inheritdoc cref="KVObject(string)"/>
+        /// <summary>
+        /// Initializes a new instance of the <see cref="KVObject"/> class with a boolean value.
+        /// </summary>
         public KVObject(bool value)
         {
             ValueType = KVValueType.Boolean;
             _scalar = value ? 1L : 0L;
         }
 
-        /// <inheritdoc cref="KVObject(string)"/>
+        /// <summary>
+        /// Initializes a new instance of the <see cref="KVObject"/> class with an integer value.
+        /// </summary>
         public KVObject(int value)
         {
             ValueType = KVValueType.Int32;
             _scalar = value;
         }
 
-        /// <inheritdoc cref="KVObject(string)"/>
+        /// <summary>
+        /// Initializes a new instance of the <see cref="KVObject"/> class with an unsigned integer value.
+        /// </summary>
         public KVObject(uint value)
         {
             ValueType = KVValueType.UInt32;
             _scalar = value;
         }
 
-        /// <inheritdoc cref="KVObject(string)"/>
+        /// <summary>
+        /// Initializes a new instance of the <see cref="KVObject"/> class with a 64-bit integer value.
+        /// </summary>
         public KVObject(long value)
         {
             ValueType = KVValueType.Int64;
             _scalar = value;
         }
 
-        /// <inheritdoc cref="KVObject(string)"/>
+        /// <summary>
+        /// Initializes a new instance of the <see cref="KVObject"/> class with an unsigned 64-bit integer value.
+        /// </summary>
         public KVObject(ulong value)
         {
             ValueType = KVValueType.UInt64;
             _scalar = unchecked((long)value);
         }
 
-        /// <inheritdoc cref="KVObject(string)"/>
+        /// <summary>
+        /// Initializes a new instance of the <see cref="KVObject"/> class with a float value.
+        /// </summary>
         public KVObject(float value)
         {
             ValueType = KVValueType.FloatingPoint;
             _scalar = BitConverter.SingleToInt32Bits(value);
         }
 
-        /// <inheritdoc cref="KVObject(string)"/>
+        /// <summary>
+        /// Initializes a new instance of the <see cref="KVObject"/> class with a double value.
+        /// </summary>
         public KVObject(double value)
         {
             ValueType = KVValueType.FloatingPoint64;
             _scalar = BitConverter.DoubleToInt64Bits(value);
         }
 
-        /// <inheritdoc cref="KVObject(string)"/>
+        /// <summary>
+        /// Initializes a new instance of the <see cref="KVObject"/> class with a pointer value.
+        /// </summary>
         public KVObject(IntPtr value)
         {
             ValueType = KVValueType.Pointer;
@@ -193,8 +210,6 @@ public KVObject this[int index]
 
         #endregion
 
-        // Operators are in KVObject_operators.cs
-
         #region Navigation
 
         /// <summary>
@@ -358,8 +373,6 @@ public void Clear()
 
         #endregion
 
-        // IEnumerable<KeyValuePair<string, KVObject>> is in KVObject_IEnumerable.cs
-
         #region Static factory methods
 
         /// <summary>
@@ -452,22 +465,7 @@ public static KVObject Blob(byte[] data)
 
         #endregion
 
-        #region Scalar access
-
-        /// <summary>Converts this value to a <see cref="bool"/>.</summary>
-        public bool ToBoolean() => ToBoolean(null);
-
-        /// <summary>Converts this value to an <see cref="int"/>.</summary>
-        public int ToInt32() => ToInt32(null);
-
-        /// <summary>Converts this value to a <see cref="long"/>.</summary>
-        public long ToInt64() => ToInt64(null);
-
-        /// <summary>Converts this value to a <see cref="float"/>.</summary>
-        public float ToSingle() => ToSingle(null);
-
-        /// <summary>Converts this value to a <see cref="double"/>.</summary>
-        public double ToDouble() => ToDouble(null);
+        #region Blob access
 
         /// <summary>Gets the binary blob data as a byte array.</summary>
         public byte[] AsBlob()
@@ -506,9 +504,6 @@ internal List<KeyValuePair<string, KVObject>> GetCollectionList()
 
         #endregion
 
-        /// <inheritdoc/>
-        public override string ToString() => ToString(CultureInfo.InvariantCulture);
-
         #region Private helpers
 
         private void SetChild(string key, KVObject value)

ValveKeyValue/ValveKeyValue/KVObject_IConvertible.cs

@@ -42,6 +42,28 @@ public partial class KVObject : IConvertible
 
         #endregion
 
+        #region Convenience overloads
+
+        /// <summary>Converts this value to a <see cref="bool"/>.</summary>
+        public bool ToBoolean() => ToBoolean(null);
+
+        /// <summary>Converts this value to an <see cref="int"/>.</summary>
+        public int ToInt32() => ToInt32(null);
+
+        /// <summary>Converts this value to a <see cref="long"/>.</summary>
+        public long ToInt64() => ToInt64(null);
+
+        /// <summary>Converts this value to a <see cref="float"/>.</summary>
+        public float ToSingle() => ToSingle(null);
+
+        /// <summary>Converts this value to a <see cref="double"/>.</summary>
+        public double ToDouble() => ToDouble(null);
+
+        /// <inheritdoc/>
+        public override string ToString() => ToString(CultureInfo.InvariantCulture);
+
+        #endregion
+
         #region Conversion methods
 
         /// <inheritdoc cref="IConvertible.ToBoolean"/>

ValveKeyValue/ValveKeyValue/KVObject_operators.cs

@@ -34,10 +34,10 @@ public partial class KVObject
         /// <summary>Implicit cast from <see cref="ushort"/> to <see cref="KVObject"/>.</summary>
         public static implicit operator KVObject(ushort value) => new(KVValueType.UInt16, (long)value);
 
-        /// <summary>Implicit cast from <see cref="byte"/> to <see cref="KVObject"/>.</summary>
+        /// <summary>Implicit cast from <see cref="byte"/> to <see cref="KVObject"/>. The value is widened to <see cref="int"/>.</summary>
         public static implicit operator KVObject(byte value) => new((int)value);
 
-        /// <summary>Implicit cast from <see cref="sbyte"/> to <see cref="KVObject"/>.</summary>
+        /// <summary>Implicit cast from <see cref="sbyte"/> to <see cref="KVObject"/>. The value is widened to <see cref="int"/>.</summary>
         public static implicit operator KVObject(sbyte value) => new((int)value);
 
         /// <summary>Implicit cast from <see cref="IntPtr"/> to <see cref="KVObject"/>.</summary>