From 5962d43f2099e2bdfc095a2cb6e8b5145a971a28 Mon Sep 17 00:00:00 2001
From: Abhinav Sarkar <abhinav@abhinavsarkar.net>
Date: Fri, 5 Jul 2019 12:38:31 +0530
Subject: [PATCH] Fix linter warnings

---
 .../java/net/abhinavsarkar/algorist/AVLTree.java   | 14 +++++---------
 1 file changed, 5 insertions(+), 9 deletions(-)

diff --git a/src/main/java/net/abhinavsarkar/algorist/AVLTree.java b/src/main/java/net/abhinavsarkar/algorist/AVLTree.java
index 91e2077..2d44854 100644
--- a/src/main/java/net/abhinavsarkar/algorist/AVLTree.java
+++ b/src/main/java/net/abhinavsarkar/algorist/AVLTree.java
@@ -220,20 +220,16 @@ public class AVLTree<K extends Comparable<K>,V> implements Iterable<AVLTree.Entr
             int balance = this.balance();
             if (balance > 1) {
                 ValueNode<K, V> l = this.left.toValueNode();
-                if (l.balance() > 0) { // left-left case
-                    return this.rotateRight();
-                } else { // left-right case
+                if (l.balance() <= 0) { // left-right case
                     this.left = l.rotateLeft();
-                    return this.rotateRight();
                 }
+                return this.rotateRight();
             } else if (balance < -1) {
                 ValueNode<K, V> r = this.right.toValueNode();
                 if (r.balance() > 0) { // right-left case
                     this.right = r.rotateRight();
-                    return this.rotateLeft();
-                } else { // right-right case
-                    return this.rotateLeft();
                 }
+                return this.rotateLeft();
             }
             return this;
         }
@@ -467,8 +463,8 @@ public class AVLTree<K extends Comparable<K>,V> implements Iterable<AVLTree.Entr
     }
 
     public static class Entry<K, V> {
-        private K key;
-        private V val;
+        private final K key;
+        private final V val;
 
         public Entry(K key, V val) {
             this.key = key;