code conventions

removed trailing spaces/empty lines
minor format

Author: simotripo

src/main/java/org/mybatis/spring/MyBatisExceptionTranslator.java

@@ -79,4 +79,5 @@ public DataAccessException translateExceptionIfPossible(RuntimeException e) {
    private void initExceptionTranslator() {
        this.exceptionTranslator = new SQLErrorCodeSQLExceptionTranslator(this.dataSource);
    }
+
 }

src/main/java/org/mybatis/spring/SqlSessionFactoryBean.java

@@ -217,7 +217,7 @@ public void afterPropertiesSet() throws Exception {
         Assert.notNull(sqlSessionFactoryBuilder, "Property 'sqlSessionFactoryBuilder' is required");
         Assert.notNull(transactionFactoryClass, "Property 'transactionFactoryClass' is required");
 
-        sqlSessionFactory = buildSqlSessionFactory();
+        this.sqlSessionFactory = buildSqlSessionFactory();
     }
 
     /**
@@ -246,10 +246,11 @@ protected SqlSessionFactory buildSqlSessionFactory() throws IOException, Illegal
                 reader = new InputStreamReader(this.configLocation.getInputStream());
                 // Null environment causes the configuration to use the default.
                 // This will be overwritten below regardless.
-                xmlConfigBuilder = new XMLConfigBuilder(reader, null, configurationProperties);
+                xmlConfigBuilder = new XMLConfigBuilder(reader, null, this.configurationProperties);
                 configuration = xmlConfigBuilder.parse();
             } catch (IOException ex) {
-                throw new NestedIOException("Failed to parse config resource: " + this.configLocation, ex);
+                throw new NestedIOException("Failed to parse config resource: "
+                        + this.configLocation, ex);
             } finally {
                 if (reader != null) {
                     try {
@@ -261,7 +262,9 @@ protected SqlSessionFactory buildSqlSessionFactory() throws IOException, Illegal
             }
 
             if (this.logger.isDebugEnabled()) {
-                this.logger.debug("Parsed configuration file: '" + this.configLocation + "'");
+                this.logger.debug("Parsed configuration file: '"
+                        + this.configLocation
+                        + "'");
             }
         } else {
             if (this.logger.isDebugEnabled()) {
@@ -309,7 +312,9 @@ protected SqlSessionFactory buildSqlSessionFactory() throws IOException, Illegal
                     XMLMapperBuilder xmlMapperBuilder = new XMLMapperBuilder(reader, configuration, path, sqlFragments);
                     xmlMapperBuilder.parse();
                 } catch (Exception e) {
-                    throw new NestedIOException("Failed to parse mapping resource: '" + mapperLocation + "'", e);
+                    throw new NestedIOException("Failed to parse mapping resource: '"
+                            + mapperLocation
+                            + "'", e);
                 } finally {
                     if (reader != null) {
                         try {
@@ -320,7 +325,9 @@ protected SqlSessionFactory buildSqlSessionFactory() throws IOException, Illegal
                 }
 
                 if (this.logger.isDebugEnabled()) {
-                    this.logger.debug("Parsed mapper file: '" + mapperLocation + "'");
+                    this.logger.debug("Parsed mapper file: '"
+                            + mapperLocation
+                            + "'");
                 }
             }
         } else {

src/main/java/org/mybatis/spring/SqlSessionTemplate.java

@@ -330,8 +330,7 @@ public Connection getConnection() {
      * Proxy needed to route MyBatis method calls to the proper SqlSession got
      * from String's Transaction Manager
      * It also unwraps exceptions thrown by {@link Method#invoke(Object, Object...)} to
-     * pass a {@link PersistenceException} to the {@link PersistenceExceptionTranslator}
-     * 
+     * pass a {@link PersistenceException} to the {@link PersistenceExceptionTranslator}.
      */
     private class SqlSessionInterceptor implements InvocationHandler {
         public Object invoke(Object proxy, Method method, Object[] args) throws Throwable {

src/main/java/org/mybatis/spring/mapper/MapperScannerConfigurer.java

@@ -202,42 +202,44 @@ public boolean match(MetadataReader metadataReader, MetadataReaderFactory metada
         @Override
         protected Set<BeanDefinitionHolder> doScan(String... basePackages) {
             Set<BeanDefinitionHolder> beanDefinitions = super.doScan(basePackages);
-            
+
             if (beanDefinitions.isEmpty()) {
                 if (logger.isDebugEnabled()) {
                     logger.debug("No MyBatis mapper was found in '"
                             + MapperScannerConfigurer.this.basePackage
                             + "' package. Please check your configuration");
                 }
-            } else {            
+            } else {
                 for (BeanDefinitionHolder holder : beanDefinitions) {
                     ScannedGenericBeanDefinition definition = (ScannedGenericBeanDefinition) holder.getBeanDefinition();
-    
+
                     if (logger.isDebugEnabled()) {
-                        logger.debug("Creating MapperFactoryBean with '" 
-                                + holder.getBeanName() + "' name and '" 
-                                + definition.getBeanClassName() + "' mapperInterface");
+                        logger.debug("Creating MapperFactoryBean with '"
+                                + holder.getBeanName()
+                                + "' name and '"
+                                + definition.getBeanClassName()
+                                + "' mapperInterface");
                     }
-                    
+
                     // the mapper interface is the original class of the bean
                     // but, the actual class of the bean is MapperFactoryBean
                     definition.getPropertyValues().add("mapperInterface", definition.getBeanClassName());
                     definition.setBeanClass(MapperFactoryBean.class);
-    
+
                     definition.getPropertyValues().add("addToConfig", MapperScannerConfigurer.this.addToConfig);
-    
+
                     if (MapperScannerConfigurer.this.sqlSessionFactory != null) {
                         definition.getPropertyValues().add("sqlSessionFactory",
                                 MapperScannerConfigurer.this.sqlSessionFactory);
                     }
-    
+
                     if (MapperScannerConfigurer.this.sqlSessionTemplate != null) {
                         definition.getPropertyValues().add("sqlSessionTemplate",
                                 MapperScannerConfigurer.this.sqlSessionTemplate);
                     }
                 }
             }
-            
+
             return beanDefinitions;
         }
 

src/main/java/org/mybatis/spring/support/SqlSessionDaoSupport.java

@@ -23,10 +23,10 @@
 import org.springframework.util.Assert;
 
 /**
- * Convenient super class for MyBatis SqlSession data access objects. 
+ * Convenient super class for MyBatis SqlSession data access objects.
  * It gives you access to the template which can then be used to execute SQL methods.
  * <p>
- * This class needs a SqlSessionTemplate or a SqlSessionFactory. 
+ * This class needs a SqlSessionTemplate or a SqlSessionFactory.
  * If both are set the SqlSessionFactory will be ignored.
  *
  * @see #setSqlSessionFactory

src/main/java/org/mybatis/spring/transaction/SpringManagedTransaction.java

@@ -31,12 +31,12 @@
 import org.springframework.util.Assert;
 
 /**
- * MyBatis has two TransactionManagers out of the box: The {@link JdbcTransaction} and the 
+ * MyBatis has two TransactionManagers out of the box: The {@link JdbcTransaction} and the
  * {@link ManagedTransaction}. When MyBatis runs under a Spring transaction none of them
  * will work well because {@link JdbcTransaction} would commit/rollback/close and it should not.
  * And {@link ManagedTransaction} would close the connection and it should not.
  * {@link SpringManagedTransaction} looks if the current connection is been managed by Spring. In that case
- * it will not commit/rollback/close. Otherwise it will behave like {@link JdbcTransaction}
+ * it will not commit/rollback/close. Otherwise it will behave like {@link JdbcTransaction}.
  *
  * @version $Id$
  */