From cc6f77efa58a25cf4401f495da77bea2433d42f4 Mon Sep 17 00:00:00 2001
From: noisyfox <timemanager.rick@gmail.com>
Date: Tue, 6 Dec 2016 22:34:17 +1100
Subject: [PATCH] Refine code

---
 .../Controller/Service/UpdateChecker.cs          | 16 +++++-----------
 1 file changed, 5 insertions(+), 11 deletions(-)

diff --git a/shadowsocks-csharp/Controller/Service/UpdateChecker.cs b/shadowsocks-csharp/Controller/Service/UpdateChecker.cs
index 83c59493..68c66279 100644
--- a/shadowsocks-csharp/Controller/Service/UpdateChecker.cs
+++ b/shadowsocks-csharp/Controller/Service/UpdateChecker.cs
@@ -25,7 +25,6 @@ namespace Shadowsocks.Controller
         public event EventHandler CheckUpdateCompleted;
 
         public const string Version = "3.3.5";
-        public const bool PreRelease = false;
 
         private class CheckUpdateTimer : System.Timers.Timer
         {
@@ -96,7 +95,7 @@ namespace Shadowsocks.Controller
                             if (ass != null)
                             {
                                 ass.prerelease = isPreRelease;
-                                if (ass.IsNewVersion(Version, PreRelease, config.checkPreRelease))
+                                if (ass.IsNewVersion(Version, config.checkPreRelease))
                                 {
                                     asserts.Add(ass);
                                 }
@@ -188,9 +187,9 @@ namespace Shadowsocks.Controller
             public string browser_download_url;
             public string suffix;
 
-            public static Asset ParseAsset(JObject aJObject)
+            public static Asset ParseAsset(JObject assertJObject)
             {
-                var name = (string) aJObject["name"];
+                var name = (string) assertJObject["name"];
                 Match match = Regex.Match(name, @"^Shadowsocks-(?<version>\d+(?:\.\d+)*)(?:|-(?<suffix>.+))\.\w+$",
                     RegexOptions.IgnoreCase);
                 if (match.Success)
@@ -199,7 +198,7 @@ namespace Shadowsocks.Controller
 
                     var asset = new Asset
                     {
-                        browser_download_url = (string) aJObject["browser_download_url"],
+                        browser_download_url = (string) assertJObject["browser_download_url"],
                         name = name,
                         version = version
                     };
@@ -215,7 +214,7 @@ namespace Shadowsocks.Controller
                 return null;
             }
 
-            public bool IsNewVersion(string currentVersion, bool isPreRelease, bool checkPreRelease)
+            public bool IsNewVersion(string currentVersion, bool checkPreRelease)
             {
                 if (prerelease && !checkPreRelease)
                 {
@@ -226,11 +225,6 @@ namespace Shadowsocks.Controller
                     return false;
                 }
                 var cmp = CompareVersion(version, currentVersion);
-                if (cmp == 0)
-                {
-                    // If current version is pre-release and we find non-prerelease version online, then the non-prelease version is newer.
-                    return !prerelease && isPreRelease;
-                }
                 return cmp > 0;
             }