Code review

JustArchiNET · Feb 24, 2016 · 44a595c · 44a595c
1 parent a528dee
commit 44a595c
Show file tree

Hide file tree

Showing 7 changed files with 115 additions and 126 deletions.
diff --git a/ArchiSteamFarm/ArchiHandler.cs b/ArchiSteamFarm/ArchiHandler.cs
@@ -205,8 +205,10 @@ internal AsyncJob<PurchaseResponseCallback> RedeemKey(string key) {
 				return null;
 			}
 
-			var request = new ClientMsgProtobuf<CMsgClientRegisterKey>(EMsg.ClientRegisterKey);
-			request.SourceJobID = Client.GetNextJobID();
+			var request = new ClientMsgProtobuf<CMsgClientRegisterKey>(EMsg.ClientRegisterKey) {
+				SourceJobID = Client.GetNextJobID()
+			};
+
 			request.Body.key = key;
 
 			Client.Send(request);
@@ -222,7 +224,7 @@ _   _                    _  _
 
 		*/
 
-		public sealed override void HandleMsg(IPacketMsg packetMsg) {
+		public override void HandleMsg(IPacketMsg packetMsg) {
 			if (packetMsg == null) {
 				return;
 			}
@@ -246,10 +248,6 @@ private void HandleFSOfflineMessageNotification(IPacketMsg packetMsg) {
 			}
 
 			var response = new ClientMsgProtobuf<CMsgClientOfflineMessageNotification>(packetMsg);
-			if (response == null) {
-				return;
-			}
-
 			Client.PostCallback(new OfflineMessageCallback(packetMsg.TargetJobID, response.Body));
 		}
 
@@ -259,10 +257,6 @@ private void HandlePurchaseResponse(IPacketMsg packetMsg) {
 			}
 
 			var response = new ClientMsgProtobuf<CMsgClientPurchaseResponse>(packetMsg);
-			if (response == null) {
-				return;
-			}
-
 			Client.PostCallback(new PurchaseResponseCallback(packetMsg.TargetJobID, response.Body));
 		}
 
@@ -272,10 +266,6 @@ private void HandleUserNotifications(IPacketMsg packetMsg) {
 			}
 
 			var response = new ClientMsgProtobuf<CMsgClientUserNotifications>(packetMsg);
-			if (response == null) {
-				return;
-			}
-
 			Client.PostCallback(new NotificationsCallback(packetMsg.TargetJobID, response.Body));
 		}
 	}