Code cleanup

Author: yfakariya

src/MsgPack/ByteBufferAllocator.cs

@@ -19,7 +19,6 @@
 #endregion -- License Terms --
 
 using System;
-using System.Collections.Generic;
 
 namespace MsgPack
 {
@@ -30,36 +29,6 @@ internal abstract class ByteBufferAllocator
 	{
 		// This type should be public when the design is fixed for Span<byte>
 
-		public abstract bool TryAllocate( byte[] oldBuffer, int requestSize, out /* Span<byte> */byte[] newBuffer );
+		public abstract bool TryAllocate( byte[] oldBuffer, int requestSize, out byte[] newBuffer );
 	}
-
-#warning TODO: SpanBufferAllocator
-    //	internal sealed class SpanBufferAllocator : ByteBufferAllocator
-    //	{
-    //		private readonly Func<int, Span<byte>> _allocator;
-
-    //		public SpanBufferAllocator( Func<int, Span<byte>> allocator)
-    //		{
-    //			this._allocator = allocator;
-    //		}
-
-    //		public override bool TryAllocate( IList<ArraySegment<byte>> arrayBuffers, IList<Span<byte>> spanBuffers, int sizeHint, ref int newCurrentBufferIndex, out Span<byte> newCurrentBuffer )
-    //		{
-    //#if DEBUG
-    //			Contract.Assert( spanBuffers != null, "spanBuffers != null" );
-    //			Contract.Assert( !spanBuffers.IsReadOnly, "!spanBuffers.IsReadOnly" );
-    //#endif // DEBUG
-    //			if ( spanBuffers.Count == Int32.MaxValue )
-    //			{
-    //				newCurrentBuffer = default( Span<byte> );
-    //				return false;
-    //			}
-
-    //			var newBuffer = this._allocator( sizeHint );
-    //			spanBuffers.Add( newBuffer );
-    //			newCurrentBuffer = newBuffer;
-    //			newCurrentBufferIndex = spanBuffers.Count - 1;
-    //			return true;
-    //		}
-    //	}
 }

src/MsgPack/MessagePackByteArrayPacker.cs

@@ -36,11 +36,9 @@ internal sealed partial class MessagePackByteArrayPacker : ByteArrayPacker
 	{
 		private const int MaximumUtf8Length = 4;
 
-		// TODO: Use Span<byte>
 		private byte[] _buffer;
 		private int _offset;
 
-		// TODO: Use Span<byte>;
 		private readonly int _initialOffset;
 		private readonly ByteBufferAllocator _allocator;