Remove unnecessarily nested else clauses

Change-Id: I04f601d2a3bb8a80de7f2ee44cae73eba61ae286

gitiles-servlet/src/main/java/com/google/gitiles/RevisionParser.java

diff --git a/gitiles-servlet/src/main/java/com/google/gitiles/RevisionParser.java b/gitiles-servlet/src/main/java/com/google/gitiles/RevisionParser.java
index de2a06f..f837bca 100644
--- a/gitiles-servlet/src/main/java/com/google/gitiles/RevisionParser.java
+++ b/gitiles-servlet/src/main/java/com/google/gitiles/RevisionParser.java
@@ -132,13 +132,12 @@
             String oldName = b.toString();
             if (!isValidRevision(oldName)) {
               return null;
-            } else {
-              RevObject old = resolve(oldName, walk);
-              if (old == null) {
-                return null;
-              }
-              oldRevision = Revision.peel(oldName, old, walk);
             }
+            RevObject old = resolve(oldName, walk);
+            if (old == null) {
+              return null;
+            }
+            oldRevision = Revision.peel(oldName, old, walk);
             part = part.substring(dots + 2);
             b = new StringBuilder();
           } else if (firstParent > 0) {
@@ -228,8 +227,7 @@
     }
     if (result.getOldRevision() != null && result.getOldRevision() != Revision.NULL) {
       return cache.isVisible(repo, walk, access, result.getOldRevision().getId(), id);
-    } else {
-      return true;
     }
+    return true;
   }
 }