Improve error routine

This way we avoid slow reflection-based Enum.IsDefined() while covering out-of-range values.
JustArchiNET · Dec 1, 2017 · 97c1a7a · 97c1a7a
1 parent d07d824
commit 97c1a7a
Showing 1 changed file with 15 additions and 3 deletions.
diff --git a/ArchiSteamFarm/ArchiHandler.cs b/ArchiSteamFarm/ArchiHandler.cs
@@ -273,9 +273,21 @@ internal NotificationsCallback(JobID jobID, CMsgClientUserNotifications msg) {
 				foreach (CMsgClientUserNotifications.Notification notification in msg.notifications) {
 					ENotification type = (ENotification) notification.user_notification_type;
 
-					if (type == ENotification.Unknown) {
-						ASF.ArchiLogger.LogGenericWarning(string.Format(Strings.WarningUnknownValuePleaseReport, nameof(type), type));
-						continue;
+					switch (type) {
+						case ENotification.AccountAlerts:
+						case ENotification.Chat:
+						case ENotification.Comments:
+						case ENotification.GameTurns:
+						case ENotification.Gifts:
+						case ENotification.HelpRequestReplies:
+						case ENotification.Invites:
+						case ENotification.Items:
+						case ENotification.ModeratorMessages:
+						case ENotification.Trading:
+							break;
+						default:
+							ASF.ArchiLogger.LogGenericWarning(string.Format(Strings.WarningUnknownValuePleaseReport, nameof(type), type));
+							continue;
 					}
 
 					Notifications[type] = notification.count;