remove unuse annotations

src/modules/registrypreview/RegistryPreviewUILib/ParseHelper.cs

@@ -92,6 +92,8 @@ public static string StripFirstAndLast(string line)
             return line;
         }
 
+        // special case for when the registryLine begins with a @ - make some tweaks and
+        // let the regular processing handle the rest.
         public static string ProcessRegistryLine(string registryLine)
         {
             if (registryLine.StartsWith("@=-", StringComparison.InvariantCulture))

src/modules/registrypreview/RegistryPreviewUILib/RegistryPreviewMainPage.Utilities.cs

@@ -227,18 +227,6 @@ private bool ParseRegistryFile(string filenameText)
                 // let the regular processing handle the rest.
                 registryLine = ParseHelper.ProcessRegistryLine(registryLine);
 
-                /* if (registryLine.StartsWith("@=-", StringComparison.InvariantCulture))
-                {
-                    // REG file has a callout to delete the @ Value which won't work *but* the Registry Editor will
-                    // clear the value of the @ Value instead, so it's still a valid line.
-                    registryLine = registryLine.Replace("@=-", "\"(Default)\"=\"\"");
-                }
-                else if (registryLine.StartsWith("@=", StringComparison.InvariantCulture))
-                {
-                    // This is the Value called "(Default)" so we tweak the line for the UX
-                    registryLine = registryLine.Replace("@=", "\"(Default)\"=");
-                }*/
-
                 // continue until we have nothing left to read
                 // switch logic, based off what the current line we're reading is
                 if (registryLine.StartsWith("[-", StringComparison.InvariantCulture))
@@ -1037,34 +1025,6 @@ private void SetValueToolTip(RegistryValue registryValue)
             registryValue.ToolTipText = value;
         }
 
-        /// <summary>
-        /// Checks a Key line for the closing bracket and treat it as an error if it cannot be found
-        /// </summary>
-        /* private void CheckKeyLineForBrackets(ref string registryLine, ref string imageName)
-        {
-            // following the current behavior of the registry editor, find the last ] and treat everything else as ignorable
-            int lastBracket = registryLine.LastIndexOf(']');
-            if (lastBracket == -1)
-            {
-                // since we don't have a last bracket yet, add an extra space and continue processing
-                registryLine += " ";
-                imageName = ERRORIMAGE;
-            }
-            else
-            {
-                // having found the last ] and there is text after it, drop the rest of the string on the floor
-                if (lastBracket < registryLine.Length - 1)
-                {
-                    registryLine = registryLine.Substring(0, lastBracket + 1);
-                }
-
-                if (CheckForKnownGoodBranches(registryLine) == false)
-                {
-                    imageName = ERRORIMAGE;
-                }
-            }
-        } */
-
         /// <summary>
         /// Takes a binary registry value, sees if it has a ; and dumps the rest of the line - this does not work for REG_SZ values
         /// </summary>
@@ -1081,41 +1041,6 @@ private string ScanAndRemoveComments(string value)
             return value.TrimEnd();
         }
 
-        /// <summary>
-        /// Make sure the root of a full path start with one of the five "hard coded" roots.  Throw an error for the branch if it doesn't.
-        /// </summary>
-        /* private bool CheckForKnownGoodBranches(string key)
-        {
-            if ((key.StartsWith("[HKEY_CLASSES_ROOT]", StringComparison.InvariantCultureIgnoreCase) == false &&
-                key.StartsWith("[HKEY_CURRENT_USER]", StringComparison.InvariantCultureIgnoreCase) == false &&
-                key.StartsWith("[HKEY_USERS]", StringComparison.InvariantCultureIgnoreCase) == false &&
-                key.StartsWith("[HKEY_LOCAL_MACHINE]", StringComparison.InvariantCultureIgnoreCase) == false &&
-                key.StartsWith("[HKEY_CURRENT_CONFIG]", StringComparison.InvariantCultureIgnoreCase) == false)
-                &&
-                (key.StartsWith(@"[HKEY_CLASSES_ROOT\", StringComparison.InvariantCultureIgnoreCase) == false &&
-                key.StartsWith(@"[HKEY_CURRENT_USER\", StringComparison.InvariantCultureIgnoreCase) == false &&
-                key.StartsWith(@"[HKEY_USERS\", StringComparison.InvariantCultureIgnoreCase) == false &&
-                key.StartsWith(@"[HKEY_LOCAL_MACHINE\", StringComparison.InvariantCultureIgnoreCase) == false &&
-                key.StartsWith(@"[HKEY_CURRENT_CONFIG\", StringComparison.InvariantCultureIgnoreCase) == false)
-                &&
-                (key.StartsWith("[HKCR]", StringComparison.InvariantCultureIgnoreCase) == false &&
-                key.StartsWith("[HKCU]", StringComparison.InvariantCultureIgnoreCase) == false &&
-                key.StartsWith("[HKU]", StringComparison.InvariantCultureIgnoreCase) == false &&
-                key.StartsWith("[HKLM]", StringComparison.InvariantCultureIgnoreCase) == false &&
-                key.StartsWith("[HKCC]", StringComparison.InvariantCultureIgnoreCase) == false)
-                &&
-                (key.StartsWith(@"[HKCR\", StringComparison.InvariantCultureIgnoreCase) == false &&
-                key.StartsWith(@"[HKCU\", StringComparison.InvariantCultureIgnoreCase) == false &&
-                key.StartsWith(@"[HKU\", StringComparison.InvariantCultureIgnoreCase) == false &&
-                key.StartsWith(@"[HKLM\", StringComparison.InvariantCultureIgnoreCase) == false &&
-                key.StartsWith(@"[HKCC\", StringComparison.InvariantCultureIgnoreCase) == false))
-            {
-                return false;
-            }
-
-            return true;
-        } */
-
         /// <summary>
         /// Turns the Open Key button in the command bar on/off, depending on if a key is selected
         /// </summary>