fix review

Author: slievrly

src/main/java/org/apache/ibatis/datasource/unpooled/UnpooledDataSource.java

@@ -292,7 +292,6 @@ public boolean jdbcCompliant() {
       return this.driver.jdbcCompliant();
     }
 
-    // @Override only valid jdk7+
     @Override
     public Logger getParentLogger() {
       return Logger.getLogger(Logger.GLOBAL_LOGGER_NAME);
@@ -309,7 +308,6 @@ public boolean isWrapperFor(Class<?> iface) throws SQLException {
     return false;
   }
 
-  // @Override only valid jdk7+
   @Override
   public Logger getParentLogger() {
     // requires JDK version 1.6

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/test/java/org/apache/ibatis/autoconstructor/ExtensiveSubject.java

@@ -62,6 +62,6 @@ public ExtensiveSubject(final byte aByte,
     }
 
     public enum TestEnum {
-        AVALUE, BVALUE, CVALUE
+        AVALUE, BVALUE, CVALUE;
     }
 }

src/test/java/org/apache/ibatis/session/SqlSessionTest.java

@@ -583,7 +583,7 @@ void shouldFailSelectOneAuthorUsingMapperClassWithTwoResultHandlers() {
     Configuration configuration = new Configuration(sqlMapper.getConfiguration().getEnvironment());
     configuration.addMapper(AuthorMapperWithMultipleHandlers.class);
     SqlSessionFactory sqlMapperWithMultipleHandlers = new DefaultSqlSessionFactory(configuration);
-    try (SqlSession sqlSession = sqlMapperWithMultipleHandlers.openSession()) {
+    try (SqlSession sqlSession = sqlMapperWithMultipleHandlers.openSession();) {
       DefaultResultHandler handler1 = new DefaultResultHandler();
       DefaultResultHandler handler2 = new DefaultResultHandler();
       AuthorMapperWithMultipleHandlers mapper = sqlSession.getMapper(AuthorMapperWithMultipleHandlers.class);
@@ -596,7 +596,7 @@ void shouldFailSelectOneAuthorUsingMapperClassWithTwoRowBounds() {
     Configuration configuration = new Configuration(sqlMapper.getConfiguration().getEnvironment());
     configuration.addMapper(AuthorMapperWithRowBounds.class);
     SqlSessionFactory sqlMapperWithMultipleHandlers = new DefaultSqlSessionFactory(configuration);
-    try (SqlSession sqlSession = sqlMapperWithMultipleHandlers.openSession()) {
+    try (SqlSession sqlSession = sqlMapperWithMultipleHandlers.openSession();) {
       RowBounds bounds1 = new RowBounds(0, 1);
       RowBounds bounds2 = new RowBounds(0, 1);
       AuthorMapperWithRowBounds mapper = sqlSession.getMapper(AuthorMapperWithRowBounds.class);