Merge pull request #1 from UpcraftLP/patch-comments

QOL changes

Author: Runemoro

src/main/java/org/dimdev/accesstransform/AccessLevel.java

@@ -1,5 +1,7 @@
 package org.dimdev.accesstransform;
 
+import java.util.Locale;
+
 public class AccessLevel {
     public enum Visibility {
         PRIVATE,
@@ -30,32 +32,13 @@ public static AccessLevel union(AccessLevel first, AccessLevel second) {
     }
 
     public static AccessLevel fromString(String string) {
+        string = string.toUpperCase(Locale.ROOT);
         boolean isFinal = true;
-        if (string.endsWith("-f")) {
+        if (string.endsWith("-F")) {
             isFinal = false;
             string = string.substring(0, string.length() - 2);
         }
 
-        switch (string) {
-            case "private": {
-                return new AccessLevel(Visibility.PRIVATE, isFinal);
-            }
-
-            case "default": {
-                return new AccessLevel(Visibility.DEFAULT, isFinal);
-            }
-
-            case "protected": {
-                return new AccessLevel(Visibility.PROTECTED, isFinal);
-            }
-
-            case "public": {
-                return new AccessLevel(Visibility.PUBLIC, isFinal);
-            }
-
-            default: {
-                throw new RuntimeException("Unknown visibility '" + string + "'");
-            }
-        }
+        return new AccessLevel(Visibility.valueOf(string), isFinal);
     }
 }

src/main/java/org/dimdev/accesstransform/AccessTransformationSet.java

@@ -15,6 +15,8 @@ public void addMimimumAccessLevel(ElementReference elementReference, AccessLevel
     }
 
     public void addMinimumAccessLevel(String string) {
+        string = string.trim();
+        if(string.isEmpty() || string.startsWith("#")) return;
         int indexOfFirstSpace = string.indexOf(' ');
         String accessLevel = string.substring(0, indexOfFirstSpace);
         String elementReference = string.substring(indexOfFirstSpace + 1);