From bcd810498395759aa3519b8a4e021c049918b2dc Mon Sep 17 00:00:00 2001 From: mmis Date: Fri, 16 Mar 2018 09:44:46 +0000 Subject: Removed checkstyle warnings Removed checkstyle warnings in policy/common/utils Issue-ID: POLICY-695 Change-Id: I831b74c889d44784d6328619d88eb92fb524ce74 Signed-off-by: mmis --- .../policy/common/utils/jpa/EntityMgrCloser.java | 58 +++++++-------- .../common/utils/jpa/EntityMgrFactoryCloser.java | 58 +++++++-------- .../policy/common/utils/jpa/EntityTransCloser.java | 83 +++++++++++----------- 3 files changed, 99 insertions(+), 100 deletions(-) (limited to 'utils/src/main/java/org') diff --git a/utils/src/main/java/org/onap/policy/common/utils/jpa/EntityMgrCloser.java b/utils/src/main/java/org/onap/policy/common/utils/jpa/EntityMgrCloser.java index 2ed54196..3532002e 100644 --- a/utils/src/main/java/org/onap/policy/common/utils/jpa/EntityMgrCloser.java +++ b/utils/src/main/java/org/onap/policy/common/utils/jpa/EntityMgrCloser.java @@ -23,37 +23,37 @@ package org.onap.policy.common.utils.jpa; import javax.persistence.EntityManager; /** - * Wrapper for an EntityManager, providing auto-close functionality. This - * is useful in try-with-resources statements. + * Wrapper for an EntityManager, providing auto-close functionality. This is useful in + * try-with-resources statements. */ public class EntityMgrCloser implements AutoCloseable { - /** - * The wrapped manager. - */ - private final EntityManager em; - - /** - * - * @param em - * manager to be auto-closed - */ - public EntityMgrCloser(EntityManager em) { - this.em = em; - } - - /** - * Gets the EntityManager wrapped within this object. - * - * @return the associated EntityManager - */ - public EntityManager getManager() { - return em; - } - - @Override - public void close() { - em.close(); - } + /** + * The wrapped manager. + */ + private final EntityManager em; + + /** + * Construct an instance with the EntityManager. + * + * @param em manager to be auto-closed + */ + public EntityMgrCloser(EntityManager em) { + this.em = em; + } + + /** + * Gets the EntityManager wrapped within this object. + * + * @return the associated EntityManager + */ + public EntityManager getManager() { + return em; + } + + @Override + public void close() { + em.close(); + } } diff --git a/utils/src/main/java/org/onap/policy/common/utils/jpa/EntityMgrFactoryCloser.java b/utils/src/main/java/org/onap/policy/common/utils/jpa/EntityMgrFactoryCloser.java index 8910c087..b046cc55 100644 --- a/utils/src/main/java/org/onap/policy/common/utils/jpa/EntityMgrFactoryCloser.java +++ b/utils/src/main/java/org/onap/policy/common/utils/jpa/EntityMgrFactoryCloser.java @@ -23,37 +23,37 @@ package org.onap.policy.common.utils.jpa; import javax.persistence.EntityManagerFactory; /** - * Wrapper for an EntityManagerFactory, providing auto-close - * functionality. This is useful in try-with-resources statements. + * Wrapper for an EntityManagerFactory, providing auto-close functionality. This is useful in + * try-with-resources statements. */ public class EntityMgrFactoryCloser implements AutoCloseable { - /** - * The wrapped factory. - */ - private final EntityManagerFactory emf; - - /** - * - * @param emf - * manager to be auto-closed - */ - public EntityMgrFactoryCloser(EntityManagerFactory emf) { - this.emf = emf; - } - - /** - * Gets the EntityManagerFactory wrapped within this object. - * - * @return the associated EntityManagerFactory - */ - public EntityManagerFactory getFactory() { - return emf; - } - - @Override - public void close() { - emf.close(); - } + /** + * The wrapped factory. + */ + private final EntityManagerFactory emf; + + /** + * Construct an instance with the given EntityManagerFactory. + * + * @param emf manager to be auto-closed + */ + public EntityMgrFactoryCloser(EntityManagerFactory emf) { + this.emf = emf; + } + + /** + * Gets the EntityManagerFactory wrapped within this object. + * + * @return the associated EntityManagerFactory + */ + public EntityManagerFactory getFactory() { + return emf; + } + + @Override + public void close() { + emf.close(); + } } diff --git a/utils/src/main/java/org/onap/policy/common/utils/jpa/EntityTransCloser.java b/utils/src/main/java/org/onap/policy/common/utils/jpa/EntityTransCloser.java index ce300741..3552a6fa 100644 --- a/utils/src/main/java/org/onap/policy/common/utils/jpa/EntityTransCloser.java +++ b/utils/src/main/java/org/onap/policy/common/utils/jpa/EntityTransCloser.java @@ -23,55 +23,54 @@ package org.onap.policy.common.utils.jpa; import javax.persistence.EntityTransaction; /** - * Wrapper for an EntityTransaction that is auto-rolled back when closed. - * This is useful in try-with-resources statements. + * Wrapper for an EntityTransaction that is auto-rolled back when closed. This is useful in + * try-with-resources statements. */ public class EntityTransCloser implements AutoCloseable { - /** - * Transaction to be rolled back. - */ - private final EntityTransaction trans; + /** + * Transaction to be rolled back. + */ + private final EntityTransaction trans; - /** - * Begins a transaction. - * - * @param et - * transaction to wrap/begin - */ - public EntityTransCloser(EntityTransaction et) { - trans = et; - trans.begin(); - } + /** + * Begins a transaction. + * + * @param et transaction to wrap/begin + */ + public EntityTransCloser(EntityTransaction et) { + trans = et; + trans.begin(); + } - /** - * Gets the wrapped transaction. - * - * @return the transaction - */ - public EntityTransaction getTransation() { - return trans; - } + /** + * Gets the wrapped transaction. + * + * @return the transaction + */ + public EntityTransaction getTransation() { + return trans; + } - /** - * Commits the transaction. - */ - public void commit() { - trans.commit(); - } + /** + * Commits the transaction. + */ + public void commit() { + trans.commit(); + } - /** - * Rolls back the transaction. - */ - public void rollback() { - trans.rollback(); - } + /** + * Rolls back the transaction. + */ + public void rollback() { + trans.rollback(); + } - @Override - public void close() { - if (trans.isActive()) { - trans.rollback(); - } - } + @Override + public void close() { + if (trans.isActive()) { + trans.rollback(); + } + } } -- cgit 1.2.3-korg