[ci] More direct returns to save on GC

Author: hazendaz

src/test/java/org/apache/ibatis/domain/blog/Author.java

@@ -134,8 +134,7 @@ public int hashCode() {
     result = 31 * result + (password != null ? password.hashCode() : 0);
     result = 31 * result + (email != null ? email.hashCode() : 0);
     result = 31 * result + (bio != null ? bio.hashCode() : 0);
-    result = 31 * result + (favouriteSection != null ? favouriteSection.hashCode() : 0);
-    return result;
+    return 31 * result + (favouriteSection != null ? favouriteSection.hashCode() : 0);
   }
 
   @Override

src/test/java/org/apache/ibatis/domain/blog/ComplexImmutableAuthor.java

@@ -70,7 +70,6 @@ public boolean equals(Object o) {
   public int hashCode() {
     int myresult = theComplexImmutableAuthorId != null ? theComplexImmutableAuthorId.hashCode() : 0;
     myresult = 31 * myresult + (bio != null ? bio.hashCode() : 0);
-    myresult = 31 * myresult + (favouriteSection != null ? favouriteSection.hashCode() : 0);
-    return myresult;
+    return 31 * myresult + (favouriteSection != null ? favouriteSection.hashCode() : 0);
   }
 }

src/test/java/org/apache/ibatis/domain/blog/ComplexImmutableAuthorId.java

@@ -76,7 +76,6 @@ public int hashCode() {
     int myresult = id;
     myresult = 31 * myresult + (email != null ? email.hashCode() : 0);
     myresult = 31 * myresult + (username != null ? username.hashCode() : 0);
-    myresult = 31 * myresult + (password != null ? password.hashCode() : 0);
-    return myresult;
+    return 31 * myresult + (password != null ? password.hashCode() : 0);
   }
 }

src/test/java/org/apache/ibatis/domain/blog/ImmutableAuthor.java

@@ -101,8 +101,7 @@ public int hashCode() {
     result = 31 * result + (password != null ? password.hashCode() : 0);
     result = 31 * result + (email != null ? email.hashCode() : 0);
     result = 31 * result + (bio != null ? bio.hashCode() : 0);
-    result = 31 * result + (favouriteSection != null ? favouriteSection.hashCode() : 0);
-    return result;
+    return 31 * result + (favouriteSection != null ? favouriteSection.hashCode() : 0);
   }
 
   @Override

src/test/java/org/apache/ibatis/domain/blog/PostLite.java

@@ -67,7 +67,6 @@ public boolean equals(Object o) {
   @Override
   public int hashCode() {
     int myresult = theId != null ? theId.hashCode() : 0;
-    myresult = 31 * myresult + blogId;
-    return myresult;
+    return 31 * myresult + blogId;
   }
 }

src/test/java/org/apache/ibatis/domain/jpetstore/Cart.java

@@ -64,10 +64,9 @@ public Item removeItemById(String itemId) {
     CartItem cartItem = itemMap.remove(itemId);
     if (cartItem == null) {
       return null;
-    } else {
-      itemList.remove(cartItem);
-      return cartItem.getItem();
     }
+    itemList.remove(cartItem);
+    return cartItem.getItem();
   }
 
   public void incrementQuantityByItemId(String itemId) {

src/test/java/org/apache/ibatis/domain/misc/CustomBeanWrapperFactory.java

@@ -25,9 +25,8 @@ public class CustomBeanWrapperFactory implements ObjectWrapperFactory {
   public boolean hasWrapperFor(Object object) {
     if (object instanceof Author) {
       return true;
-    } else {
-      return false;
     }
+    return false;
   }
 
   @Override

src/test/java/org/apache/ibatis/submitted/cglib_lazy_error/Person.java

@@ -25,9 +25,8 @@ public class Person {
   public Person getAncestor() {
     if (getParent() == null) {
       return this;
-    } else {
-      return getParent().getAncestor();
     }
+    return getParent().getAncestor();
   }
 
   @Override

src/test/java/org/apache/ibatis/submitted/column_prefix/ColumnPrefixAutoMappingTest.java

@@ -25,15 +25,12 @@
 class ColumnPrefixAutoMappingTest extends ColumnPrefixTest {
   @Override
   protected List<Pet> getPetAndRoom(SqlSession sqlSession) {
-    List<Pet> pets = sqlSession.selectList("org.apache.ibatis.submitted.column_prefix.MapperAutoMapping.selectPets");
-    return pets;
+    return sqlSession.selectList("org.apache.ibatis.submitted.column_prefix.MapperAutoMapping.selectPets");
   }
 
   @Override
   protected List<Person> getPersons(SqlSession sqlSession) {
-    List<Person> list = sqlSession
-        .selectList("org.apache.ibatis.submitted.column_prefix.MapperAutoMapping.selectPersons");
-    return list;
+    return sqlSession.selectList("org.apache.ibatis.submitted.column_prefix.MapperAutoMapping.selectPersons");
   }
 
   @Override

src/test/java/org/apache/ibatis/submitted/column_prefix/ColumnPrefixNestedQueryTest.java

@@ -22,15 +22,12 @@
 class ColumnPrefixNestedQueryTest extends ColumnPrefixTest {
   @Override
   protected List<Pet> getPetAndRoom(SqlSession sqlSession) {
-    List<Pet> pets = sqlSession.selectList("org.apache.ibatis.submitted.column_prefix.MapperNestedQuery.selectPets");
-    return pets;
+    return sqlSession.selectList("org.apache.ibatis.submitted.column_prefix.MapperNestedQuery.selectPets");
   }
 
   @Override
   protected List<Person> getPersons(SqlSession sqlSession) {
-    List<Person> list = sqlSession
-        .selectList("org.apache.ibatis.submitted.column_prefix.MapperNestedQuery.selectPersons");
-    return list;
+    return sqlSession.selectList("org.apache.ibatis.submitted.column_prefix.MapperNestedQuery.selectPersons");
   }
 
   @Override