Merge pull request #1662 from paopaofish/master

Remove several redundant operations

Author: kazuki43zoo

src/main/java/org/apache/ibatis/mapping/ResultMapping.java

@@ -262,11 +262,7 @@ public boolean equals(Object o) {
 
     ResultMapping that = (ResultMapping) o;
 
-    if (property == null || !property.equals(that.property)) {
-      return false;
-    }
-
-    return true;
+    return property != null && property.equals(that.property);
   }
 
   @Override

src/main/java/org/apache/ibatis/mapping/SqlCommandType.java

@@ -19,5 +19,5 @@
  * @author Clinton Begin
  */
 public enum SqlCommandType {
-  UNKNOWN, INSERT, UPDATE, DELETE, SELECT, FLUSH;
+  UNKNOWN, INSERT, UPDATE, DELETE, SELECT, FLUSH
 }

src/main/java/org/apache/ibatis/session/defaults/DefaultSqlSession.java

@@ -269,7 +269,7 @@ public void close() {
   }
 
   private void closeCursors() {
-    if (cursorList != null && cursorList.size() != 0) {
+    if (cursorList != null && !cursorList.isEmpty()) {
       for (Cursor<?> cursor : cursorList) {
         try {
           cursor.close();