Fixed all warnings

DarkRiftNetworking · Dec 31, 2016 · 7f21d42 · 7f21d42
1 parent 382f24d
commit 7f21d42
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 11 deletions.
diff --git a/Hazel.UnitTests/TcpConnectionTests.cs b/Hazel.UnitTests/TcpConnectionTests.cs
@@ -19,7 +19,7 @@ public void TcpFieldTest()
         {
             NetworkEndPoint ep = new NetworkEndPoint(IPAddress.Loopback, 4296);
 
-            using (TcpConnectionListener listener = new TcpConnectionListener(IPAddress.Any, 4296))
+            using (TcpConnectionListener listener = new TcpConnectionListener(new NetworkEndPoint(IPAddress.Any, 4296)))
             using (TcpConnection connection = new TcpConnection(ep))
             {
                 listener.Start();
@@ -39,7 +39,7 @@ public void TcpFieldTest()
         [TestMethod]
         public void TcpHandshakeTest()
         {
-            using (TcpConnectionListener listener = new TcpConnectionListener(IPAddress.Any, 4296, IPMode.IPv4))
+            using (TcpConnectionListener listener = new TcpConnectionListener(new NetworkEndPoint(IPAddress.Any, 4296, IPMode.IPv4)))
             using (TcpConnection connection = new TcpConnection(new NetworkEndPoint(IPAddress.Loopback, 4296, IPMode.IPv4)))
             {
                 listener.Start();
@@ -59,7 +59,7 @@ public void TcpHandshakeTest()
         [TestMethod]
         public void TcpIPv4ConnectionTest()
         {
-            using (TcpConnectionListener listener = new TcpConnectionListener(IPAddress.Any, 4296, IPMode.IPv4))
+            using (TcpConnectionListener listener = new TcpConnectionListener(new NetworkEndPoint(IPAddress.Any, 4296, IPMode.IPv4)))
             using (TcpConnection connection = new TcpConnection(new NetworkEndPoint(IPAddress.Loopback, 4296, IPMode.IPv4)))
             {
                 listener.Start();
@@ -74,7 +74,7 @@ public void TcpIPv4ConnectionTest()
         [TestMethod]
         public void TcpIPv6ConnectionTest()
         {
-            using (TcpConnectionListener listener = new TcpConnectionListener(IPAddress.IPv6Any, 4296, IPMode.IPv6))
+            using (TcpConnectionListener listener = new TcpConnectionListener(new NetworkEndPoint(IPAddress.IPv6Any, 4296, IPMode.IPv6)))
             {
                 listener.Start();
 
@@ -91,7 +91,7 @@ public void TcpIPv6ConnectionTest()
         [TestMethod]
         public void TcpServerToClientTest()
         {
-            using (TcpConnectionListener listener = new TcpConnectionListener(IPAddress.Any, 4296))
+            using (TcpConnectionListener listener = new TcpConnectionListener(new NetworkEndPoint(IPAddress.Any, 4296)))
             using (TcpConnection connection = new TcpConnection(new NetworkEndPoint(IPAddress.Loopback, 4296)))
             {
                 TestHelper.RunServerToClientTest(listener, connection, 10, SendOption.FragmentedReliable);
@@ -104,7 +104,7 @@ public void TcpServerToClientTest()
         [TestMethod]
         public void TcpClientToServerTest()
         {
-            using (TcpConnectionListener listener = new TcpConnectionListener(IPAddress.Any, 4296))
+            using (TcpConnectionListener listener = new TcpConnectionListener(new NetworkEndPoint(IPAddress.Any, 4296)))
             using (TcpConnection connection = new TcpConnection(new NetworkEndPoint(IPAddress.Loopback, 4296)))
             {
                 TestHelper.RunClientToServerTest(listener, connection, 10, SendOption.FragmentedReliable);
@@ -117,7 +117,7 @@ public void TcpClientToServerTest()
         [TestMethod]
         public void ClientDisconnectTest()
         {
-            using (TcpConnectionListener listener = new TcpConnectionListener(IPAddress.Any, 4296))
+            using (TcpConnectionListener listener = new TcpConnectionListener(new NetworkEndPoint(IPAddress.Any, 4296)))
             using (TcpConnection connection = new TcpConnection(new NetworkEndPoint(IPAddress.Loopback, 4296)))
             {
                 TestHelper.RunClientDisconnectTest(listener, connection);
@@ -130,7 +130,7 @@ public void ClientDisconnectTest()
         [TestMethod]
         public void ServerDisconnectTest()
         {
-            using (TcpConnectionListener listener = new TcpConnectionListener(IPAddress.Any, 4296))
+            using (TcpConnectionListener listener = new TcpConnectionListener(new NetworkEndPoint(IPAddress.Any, 4296)))
             using (TcpConnection connection = new TcpConnection(new NetworkEndPoint(IPAddress.Loopback, 4296)))
             {
                 TestHelper.RunServerDisconnectTest(listener, connection);

diff --git a/Hazel.UnitTests/UdpConnectionTests.cs b/Hazel.UnitTests/UdpConnectionTests.cs
@@ -39,7 +39,7 @@ public void UdpFieldTest()
         [TestMethod]
         public void UdpHandshakeTest()
         {
-            using (UdpConnectionListener listener = new UdpConnectionListener(IPAddress.Any, 4296, IPMode.IPv4))
+            using (UdpConnectionListener listener = new UdpConnectionListener(new NetworkEndPoint(IPAddress.Any, 4296, IPMode.IPv4)))
             using (UdpConnection connection = new UdpClientConnection(new NetworkEndPoint(IPAddress.Loopback, 4296, IPMode.IPv4)))
             {
                 listener.Start();

diff --git a/Hazel/Udp/UdpConnection.Fragmented.cs b/Hazel/Udp/UdpConnection.Fragmented.cs
@@ -27,7 +27,7 @@ partial class UdpConnection
         void FragmentedSend(byte[] data)
         {
             //Get an ID not used yet.
-            ushort id = ++lastFragmentIDAllocated;      //TODO manage loop around
+            ushort id = ++lastFragmentIDAllocated;      //TODO is extra code needed to manage loop around?
 
             for (ushort i = 0; i < Math.Ceiling(data.Length / (double)FragmentSize); i++)
             {

diff --git a/Hazel/Udp/UdpConnectionListener.cs b/Hazel/Udp/UdpConnectionListener.cs
@@ -117,7 +117,7 @@ void ReadCallback(IAsyncResult result)
                 //If the socket's been disposed then we can just end there.
                 return;
             }
-            catch (SocketException e)
+            catch (SocketException)
             {
                 //Client no longer reachable, pretend it didn't happen
                 //TODO should this not inform the connection this client is lost???