Add GetShort() method in dataTypes

MinecraftClient/MinecraftClient.csproj

@@ -289,7 +289,6 @@
     <Content Include="Protocol\Dns\Records\totla.txt" />
     <Content Include="Resources\AppIcon.ico" />
   </ItemGroup>
-  <ItemGroup />
   <Import Project="$(MSBuildToolsPath)\Microsoft.CSharp.targets" />
   <!-- To modify your build process, add your task inside one of the targets below and uncomment it. 
        Other similar extension points exist, see Microsoft.Common.targets.

MinecraftClient/MinecraftClient.csproj.user

@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="utf-8"?>
 <Project ToolsVersion="4.0" xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
   <PropertyGroup Condition="'$(Configuration)|$(Platform)' == 'Debug|x86'">
-    <StartArguments>test - 192.168.1.29</StartArguments>
+    <StartArguments>TestBot - localhost</StartArguments>
   </PropertyGroup>
   <PropertyGroup>
     <PublishUrlHistory>publish\</PublishUrlHistory>

MinecraftClient/Protocol/Handlers/DataTypes.cs

@@ -412,6 +412,18 @@ namespace MinecraftClient.Protocol.Handlers
             return bytes.ToArray();
         }
 
+        /// <summary>
+        /// Get byte array representing a short
+        /// </summary>
+        /// <param name="number">Short to process</param>
+        /// <returns>Array ready to send</returns>
+        public byte[] GetShort(short number)
+        {
+            byte[] theShort = BitConverter.GetBytes(number);
+            Array.Reverse(theShort);
+            return theShort;
+        }
+
         /// <summary>
         /// Get byte array representing a double
         /// </summary>

MinecraftClient/Protocol/Handlers/Protocol18.cs

@@ -1366,10 +1366,7 @@ namespace MinecraftClient.Protocol.Handlers
             try
             {
                 List<byte> packet = new List<byte>();
-                // short to byte (?
+                packet.AddRange(dataTypes.GetShort(slot));
-                byte[] b = BitConverter.GetBytes(slot);
-                Array.Reverse(b);
-                packet.AddRange(b);
                 SendPacket(PacketOutgoingType.HeldItemChange, packet);
                 return true;
             }