Added IBitWritable interface for structs

Author: TwoTenPvP

MLAPI/NetworkingManagerComponents/Binary/BitReader.cs

@@ -174,10 +174,10 @@ public object ReadObjectPacked(Type type)
                     return null;
                 }
             }
-            if (typeof(BitWritable).IsAssignableFrom(type))
+            if (typeof(IBitWritable).IsAssignableFrom(type))
             {
                 object instance = Activator.CreateInstance(type);
-                ((BitWritable)instance).Read(this.source);
+                ((IBitWritable)instance).Read(this.source);
                 return instance;
             }
 

MLAPI/NetworkingManagerComponents/Binary/BitWritable.cs

@@ -4,19 +4,27 @@
 
 namespace MLAPI.Serialization 
 {
-    [Obsolete("The IBitWritable interface has been replaced with the abstract BitWritable class", true)]
+    /// <summary>
+    /// Interface for serializable classes and structs
+    /// </summary>
     public interface IBitWritable
     {
-        [Obsolete("The IBitWritable interface has been replaced with the abstract BitWritable class", true)]
+        /// <summary>
+        /// Reads the contents from the stream and applies it to the type instance
+        /// </summary>
+        /// <param name="stream">The stream to read from</param>
         void Read(Stream stream);
-        [Obsolete("The IBitWritable interface has been replaced with the abstract BitWritable class", true)]
+        /// <summary>
+        /// Writes the contents of the type instance to the stream
+        /// </summary>
+        /// <param name="stream">The stream to write to</param>
         void Write(Stream stream);
     }
 
     /// <summary>
     /// BitWritable is the base class for writable types
     /// </summary>
-    public abstract class BitWritable
+    public abstract class AutoBitWritable : IBitWritable
     {        
         /// <summary>
         /// Writes the contents of the type instance to the stream

MLAPI/NetworkingManagerComponents/Binary/BitWriter.cs

@@ -190,13 +190,13 @@ public void WriteObjectPacked(object value)
                 WriteUInt16Packed(((NetworkedBehaviour)value).GetBehaviourId());
                 return;
             } 
-            else if (value is BitWritable)
+            else if (value is IBitWritable)
             {
                 if (value == null)
                 {
                     throw new ArgumentException("BitWriter cannot write IBitWritable types with a null value");
                 }
-                ((BitWritable)value).Write(this.sink);
+                ((IBitWritable)value).Write(this.sink);
                 return;
             } 
 

MLAPI/NetworkingManagerComponents/Binary/SerializationHelper.cs

@@ -2,6 +2,7 @@
 using System.Collections.Generic;
 using System.Linq;
 using System.Reflection;
+using MLAPI.Internal;
 using UnityEngine;
 
 namespace MLAPI.Serialization
@@ -65,7 +66,7 @@ internal static FieldInfo[] GetFieldsForType(Type type)
         /// <returns>Whether or not the type is supported</returns>
         public static bool IsTypeSupported(Type type)
         {
-            return type.IsEnum || SupportedTypes.Contains(type) || (typeof(BitWritable).IsAssignableFrom(type));
+            return type.IsEnum || SupportedTypes.Contains(type) || type.HasInterface(typeof(IBitWritable));
         }
     }
 }