You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openjpa.apache.org by st...@apache.org on 2015/04/06 22:57:13 UTC

svn commit: r1671694 [1/2] - in /openjpa/trunk: openjpa-examples/openbooks/src/main/java/jpa/tools/swing/ openjpa-examples/openbooks/src/main/java/openbook/domain/ openjpa-examples/openbooks/src/main/java/openbook/server/ openjpa-examples/openbooks/src...

Author: struberg
Date: Mon Apr  6 20:57:11 2015
New Revision: 1671694

URL: http://svn.apache.org/r1671694
Log:
remove empty @return javadocs

just caused javadoc warnings...

Modified:
    openjpa/trunk/openjpa-examples/openbooks/src/main/java/jpa/tools/swing/EntityDataModel.java
    openjpa/trunk/openjpa-examples/openbooks/src/main/java/jpa/tools/swing/EntityTypeView.java
    openjpa/trunk/openjpa-examples/openbooks/src/main/java/jpa/tools/swing/FileScanner.java
    openjpa/trunk/openjpa-examples/openbooks/src/main/java/jpa/tools/swing/MetamodelHelper.java
    openjpa/trunk/openjpa-examples/openbooks/src/main/java/jpa/tools/swing/MetamodelView.java
    openjpa/trunk/openjpa-examples/openbooks/src/main/java/openbook/domain/Customer.java
    openjpa/trunk/openjpa-examples/openbooks/src/main/java/openbook/domain/ShoppingCart.java
    openjpa/trunk/openjpa-examples/openbooks/src/main/java/openbook/server/OpenBookService.java
    openjpa/trunk/openjpa-examples/openbooks/src/main/java/openbook/server/OpenBookServiceImpl.java
    openjpa/trunk/openjpa-examples/openbooks/src/main/java/openbook/tools/CommandProcessor.java
    openjpa/trunk/openjpa-examples/openbooks/src/main/java/openbook/tools/converter/SourceRenderer.java
    openjpa/trunk/openjpa-examples/openbooks/src/main/java/openbook/util/Randomizer.java
    openjpa/trunk/openjpa-examples/opentrader/src/main/java/org/apache/openjpa/trader/service/TradingService.java
    openjpa/trunk/openjpa-integration/daytrader/src/test/java/org/apache/openjpa/integration/daytrader/TradeScenario.java
    openjpa/trunk/openjpa-integration/validation/src/test/java/org/apache/openjpa/integration/validation/TestConstraints.java
    openjpa/trunk/openjpa-integration/validation/src/test/java/org/apache/openjpa/integration/validation/TestMerge.java
    openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/identifier/DBIdentifier.java
    openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/identifier/DBIdentifierUtil.java
    openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/identifier/Normalizer.java
    openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/identifier/QualifiedDBIdentifier.java
    openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/meta/ReverseMappingTool.java
    openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/meta/SequenceMapping.java
    openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/schema/SchemaGroup.java
    openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/DBDictionary.java
    openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/Joins.java
    openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/PostgresDictionary.java
    openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/Select.java
    openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/SelectImpl.java
    openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/json/JSONObject.java
    openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/IOR.java
    openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/JESTContext.java
    openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/JPAServletContext.java
    openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/JSONObject.java
    openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/JSONObjectFormatter.java
    openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/MetamodelHelper.java
    openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/TokenReplacedStream.java
    openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/XMLFormatter.java
    openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/conf/Specification.java
    openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/instrumentation/AbstractQueryCacheInstrument.java
    openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/instrumentation/PreparedQueryCacheInstrument.java
    openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/instrumentation/QueryCacheInstrument.java
    openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/instrumentation/jmx/JMXInstrument.java
    openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/instrumentation/jmx/JMXProvider.java
    openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/FinderQuery.java
    openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/exps/AbstractExpressionBuilder.java
    openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/exps/Context.java
    openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/exps/QueryExpressions.java
    openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/meta/ClassMetaData.java
    openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/meta/MetaDataDefaults.java
    openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/meta/ValueMetaDataImpl.java
    openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/util/DelayedProxy.java
    openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/ConfigurationImpl.java
    openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/Configurations.java
    openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/identifier/IdentifierConfiguration.java
    openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/identifier/IdentifierUtilImpl.java
    openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/instrumentation/Instrument.java
    openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/instrumentation/InstrumentationProvider.java
    openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/callbacks/PostPersistEntity.java
    openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/common/utils/AbstractTestCase.java
    openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/datacache/TestDataCacheBehavesIdentical.java
    openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/graph/Graph.java
    openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/kernel/TestFetchPlan.java
    openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/query/TestMethodQLQuery.java
    openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/test/AbstractPersistenceTestCase.java
    openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/xmlmapping/xmlbindings/myaddress/Address.java
    openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/xmlmapping/xmlbindings/myaddress/CANAddress.java
    openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/xmlmapping/xmlbindings/myaddress/GBRAddress.java
    openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/xmlmapping/xmlbindings/myaddress/USAAddress.java
    openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/AnnotationPersistenceMetaDataSerializer.java
    openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/HintHandler.java
    openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/XMLPersistenceMetaDataParser.java
    openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/validation/ValidatorImpl.java
    openjpa/trunk/openjpa-slice/src/main/java/org/apache/openjpa/slice/SliceInfo.java
    openjpa/trunk/openjpa-slice/src/test/java/org/apache/openjpa/slice/TestQueryTargetPolicy.java
    openjpa/trunk/openjpa-tools/openjpa-maven-plugin/src/main/java/org/apache/openjpa/tools/maven/AbstractOpenJpaMojo.java

Modified: openjpa/trunk/openjpa-examples/openbooks/src/main/java/jpa/tools/swing/EntityDataModel.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-examples/openbooks/src/main/java/jpa/tools/swing/EntityDataModel.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-examples/openbooks/src/main/java/jpa/tools/swing/EntityDataModel.java (original)
+++ openjpa/trunk/openjpa-examples/openbooks/src/main/java/jpa/tools/swing/EntityDataModel.java Mon Apr  6 20:57:11 2015
@@ -195,9 +195,6 @@ public class EntityDataModel<T> extends
 
     /**
      * Gets the value by reflection.
-     * @param o
-     * @param p
-     * @return
      */
     Object getValueByReflection(int rowIndex, Object o, int columnIndex, Method m) {
         if (o == null) {

Modified: openjpa/trunk/openjpa-examples/openbooks/src/main/java/jpa/tools/swing/EntityTypeView.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-examples/openbooks/src/main/java/jpa/tools/swing/EntityTypeView.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-examples/openbooks/src/main/java/jpa/tools/swing/EntityTypeView.java (original)
+++ openjpa/trunk/openjpa-examples/openbooks/src/main/java/jpa/tools/swing/EntityTypeView.java Mon Apr  6 20:57:11 2015
@@ -67,7 +67,6 @@ public class EntityTypeView<T> extends J
     /**
      * Gets the top left position of the attribute label relative to this entity view.
      * @param attr
-     * @return
      */
     public Point getPosition(Attribute<?,?> attr) {
         for (Component c : getComponents()) {

Modified: openjpa/trunk/openjpa-examples/openbooks/src/main/java/jpa/tools/swing/FileScanner.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-examples/openbooks/src/main/java/jpa/tools/swing/FileScanner.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-examples/openbooks/src/main/java/jpa/tools/swing/FileScanner.java (original)
+++ openjpa/trunk/openjpa-examples/openbooks/src/main/java/jpa/tools/swing/FileScanner.java Mon Apr  6 20:57:11 2015
@@ -34,8 +34,6 @@ public class FileScanner {
     
     /**
      * Scans the given  
-     * @param root
-     * @return
      */
     public List<File> scan(File dir) {
         List<File> bag = new ArrayList<File>();

Modified: openjpa/trunk/openjpa-examples/openbooks/src/main/java/jpa/tools/swing/MetamodelHelper.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-examples/openbooks/src/main/java/jpa/tools/swing/MetamodelHelper.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-examples/openbooks/src/main/java/jpa/tools/swing/MetamodelHelper.java (original)
+++ openjpa/trunk/openjpa-examples/openbooks/src/main/java/jpa/tools/swing/MetamodelHelper.java Mon Apr  6 20:57:11 2015
@@ -59,7 +59,6 @@ public class MetamodelHelper {
     /**
      * Gets the displayed name of a given entity type.
      * @param type
-     * @return
      */
     public static String getDisplayName(EntityType<?> type) {
         return type.getName();
@@ -201,7 +200,6 @@ public class MetamodelHelper {
      * Gets the value of the given persistent attribute for the given instance.
      * @param attr
      * @param instance
-     * @return
      */
     public static Object getValue(Attribute<?,?> attr, Object instance) {
         Member member = attr.getJavaMember();

Modified: openjpa/trunk/openjpa-examples/openbooks/src/main/java/jpa/tools/swing/MetamodelView.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-examples/openbooks/src/main/java/jpa/tools/swing/MetamodelView.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-examples/openbooks/src/main/java/jpa/tools/swing/MetamodelView.java (original)
+++ openjpa/trunk/openjpa-examples/openbooks/src/main/java/jpa/tools/swing/MetamodelView.java Mon Apr  6 20:57:11 2015
@@ -126,7 +126,6 @@ public class MetamodelView extends JPane
      * Gets the position of the attribute in the entity view relative to this panel. 
      * @param a
      * @param attr
-     * @return
      */
     Point getConnectLocation(EntityTypeView<?> a, Attribute<?,?> attr) {
         Point p1 = a.getLocation();

Modified: openjpa/trunk/openjpa-examples/openbooks/src/main/java/openbook/domain/Customer.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-examples/openbooks/src/main/java/openbook/domain/Customer.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-examples/openbooks/src/main/java/openbook/domain/Customer.java (original)
+++ openjpa/trunk/openjpa-examples/openbooks/src/main/java/openbook/domain/Customer.java Mon Apr  6 20:57:11 2015
@@ -66,7 +66,6 @@ public class Customer implements Seriali
     
     /**
      * Create a {@link ShoppingCart} for this customer.
-     * @return
      */
     public ShoppingCart newCart() {
         return new ShoppingCart(this);

Modified: openjpa/trunk/openjpa-examples/openbooks/src/main/java/openbook/domain/ShoppingCart.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-examples/openbooks/src/main/java/openbook/domain/ShoppingCart.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-examples/openbooks/src/main/java/openbook/domain/ShoppingCart.java (original)
+++ openjpa/trunk/openjpa-examples/openbooks/src/main/java/openbook/domain/ShoppingCart.java Mon Apr  6 20:57:11 2015
@@ -62,7 +62,6 @@ public class ShoppingCart implements Ser
     /**
      * Gets the books with their corresponding quantity in this cart.
      * 
-     * @return
      */
     public Map<Book, Integer> getItems() {
         return items;

Modified: openjpa/trunk/openjpa-examples/openbooks/src/main/java/openbook/server/OpenBookService.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-examples/openbooks/src/main/java/openbook/server/OpenBookService.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-examples/openbooks/src/main/java/openbook/server/OpenBookService.java (original)
+++ openjpa/trunk/openjpa-examples/openbooks/src/main/java/openbook/server/OpenBookService.java Mon Apr  6 20:57:11 2015
@@ -55,7 +55,6 @@ public interface OpenBookService {
      * @param max maximum price
      * @param author name of author
      * @param decorators to modify the executable query such as its range.
-     * @return
      */
     public List<Book> select(String title, 
             Double min, Double max, 
@@ -71,7 +70,6 @@ public interface OpenBookService {
      * @param max maximum price
      * @param author name of author
      * @param decorators to modify the executable query such as its range.
-     * @return
      */
     public String getQuery(String title, 
             Double min, Double max, 
@@ -163,21 +161,18 @@ public interface OpenBookService {
     /**
      * Gets the underlying persistence unit.
      * 
-     * @return
      */
     public EntityManagerFactory getUnit();
     
     /**
      * Gets the name of the underlying persistence unit.
      * 
-     * @return
      */
     public String getUnitName();
     
     /**
      * Affirms if the transaction on this persistence unit is managed by a container.
      * 
-     * @return
      */
     public boolean isManaged();
     

Modified: openjpa/trunk/openjpa-examples/openbooks/src/main/java/openbook/server/OpenBookServiceImpl.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-examples/openbooks/src/main/java/openbook/server/OpenBookServiceImpl.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-examples/openbooks/src/main/java/openbook/server/OpenBookServiceImpl.java (original)
+++ openjpa/trunk/openjpa-examples/openbooks/src/main/java/openbook/server/OpenBookServiceImpl.java Mon Apr  6 20:57:11 2015
@@ -190,14 +190,8 @@ class OpenBookServiceImpl extends Persis
     
     /**
      * Find books that match title and price range.
-     * @param title
-     * @param minPrice
-     * @param maxPrice
-     * @param author
-     * @return
      */
-    
-    public List<Book> select(String title, Double minPrice, Double maxPrice, String author, 
+    public List<Book> select(String title, Double minPrice, Double maxPrice, String author,
             QueryDecorator...decorators) {
         CriteriaQuery<Book> q = buildQuery(title, minPrice, maxPrice, author);
         EntityManager em = begin();
@@ -233,7 +227,6 @@ class OpenBookServiceImpl extends Persis
      * 
      * @return a typed query
      */
-
     private CriteriaQuery<Book> buildQuery(String title, Double minPrice, Double maxPrice, String author) {
         // builder generates the Criteria Query as well as all the expressions
         CriteriaBuilder cb = getUnit().getCriteriaBuilder();

Modified: openjpa/trunk/openjpa-examples/openbooks/src/main/java/openbook/tools/CommandProcessor.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-examples/openbooks/src/main/java/openbook/tools/CommandProcessor.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-examples/openbooks/src/main/java/openbook/tools/CommandProcessor.java (original)
+++ openjpa/trunk/openjpa-examples/openbooks/src/main/java/openbook/tools/CommandProcessor.java Mon Apr  6 20:57:11 2015
@@ -323,8 +323,6 @@ public class CommandProcessor {
         
         /**
          * Affirms if the given name any of the aliases.
-         * @param alias
-         * @return
          */
         public boolean match(String name) {
             for (String alias : aliases) {

Modified: openjpa/trunk/openjpa-examples/openbooks/src/main/java/openbook/tools/converter/SourceRenderer.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-examples/openbooks/src/main/java/openbook/tools/converter/SourceRenderer.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-examples/openbooks/src/main/java/openbook/tools/converter/SourceRenderer.java (original)
+++ openjpa/trunk/openjpa-examples/openbooks/src/main/java/openbook/tools/converter/SourceRenderer.java Mon Apr  6 20:57:11 2015
@@ -137,7 +137,6 @@ public abstract class SourceRenderer {
      * 
      * @param srcDir the root source directory.
      * @param path the path to input file.
-     * @return
      */
     protected InputStream getInputStream(File srcDir, String path) {
         File file = new File(srcDir, path); 
@@ -156,7 +155,6 @@ public abstract class SourceRenderer {
     /**
      * Gets the output stream to write to.
      * @param file
-     * @return
      */
     private FileOutputStream createOutput(File file)  {
         try {

Modified: openjpa/trunk/openjpa-examples/openbooks/src/main/java/openbook/util/Randomizer.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-examples/openbooks/src/main/java/openbook/util/Randomizer.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-examples/openbooks/src/main/java/openbook/util/Randomizer.java (original)
+++ openjpa/trunk/openjpa-examples/openbooks/src/main/java/openbook/util/Randomizer.java Mon Apr  6 20:57:11 2015
@@ -91,7 +91,6 @@ public class Randomizer {
      * @param <T>
      * @param list
      * @param n
-     * @return
      */
     public static <T> List<T> selectRandom(List<T> list, int n) {
         if (list == null || list.isEmpty())

Modified: openjpa/trunk/openjpa-examples/opentrader/src/main/java/org/apache/openjpa/trader/service/TradingService.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-examples/opentrader/src/main/java/org/apache/openjpa/trader/service/TradingService.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-examples/opentrader/src/main/java/org/apache/openjpa/trader/service/TradingService.java (original)
+++ openjpa/trunk/openjpa-examples/opentrader/src/main/java/org/apache/openjpa/trader/service/TradingService.java Mon Apr  6 20:57:11 2015
@@ -84,7 +84,6 @@ public interface TradingService {
      * Logs in a trader of given name.
      * 
      * @param trader
-     * @return
      */
     Trader login(String trader);
     
@@ -94,7 +93,6 @@ public interface TradingService {
      * @param stock
      * @param volume
      * @param price
-     * @return
      */
     Ask ask(Trader trader, Stock stock, int volume, double price);
     
@@ -105,7 +103,6 @@ public interface TradingService {
      * @param stock
      * @param volume
      * @param price
-     * @return
      */
     Bid bid(Trader trader, Stock stock, int volume, double price);
     
@@ -137,7 +134,6 @@ public interface TradingService {
      * Gets the trades executed between the given time periods.
      * @param from
      * @param to
-     * @return
      */
     List<Trade> getTrades(Timestamp from, Timestamp to);
     
@@ -145,27 +141,23 @@ public interface TradingService {
      * Gets the trades executed by the given trader between the given time periods.
      * @param from
      * @param to
-     * @return
      */
     List<Trade> getTrades(Trader trader, Boolean boughtOrsold, Timestamp from, Timestamp to);
     
     /**
      * Gets the current value of the stock given its symbol.
      * @param symbol
-     * @return
      */
     Stock getStock(String symbol);
     
     /**
      * Gets the statements logged since the last call.
-     * @return
      */
     List<LogStatement> getLog();
     
     /**
      * Get a descriptive URI-like string for this service.
      * 
-     * @return
      */
     String getServiceURI();
 }

Modified: openjpa/trunk/openjpa-integration/daytrader/src/test/java/org/apache/openjpa/integration/daytrader/TradeScenario.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-integration/daytrader/src/test/java/org/apache/openjpa/integration/daytrader/TradeScenario.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-integration/daytrader/src/test/java/org/apache/openjpa/integration/daytrader/TradeScenario.java (original)
+++ openjpa/trunk/openjpa-integration/daytrader/src/test/java/org/apache/openjpa/integration/daytrader/TradeScenario.java Mon Apr  6 20:57:11 2015
@@ -39,7 +39,6 @@ public class TradeScenario {
      *     login, home, account, update, home, portfolio, sell, buy, home, portfolio, sell, buy, home, account, logout
      *     
      * @param userID
-     * @return
      */
     public boolean performUserTasks(String userID) {
         StringBuilder sb = new StringBuilder(16384);

Modified: openjpa/trunk/openjpa-integration/validation/src/test/java/org/apache/openjpa/integration/validation/TestConstraints.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-integration/validation/src/test/java/org/apache/openjpa/integration/validation/TestConstraints.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-integration/validation/src/test/java/org/apache/openjpa/integration/validation/TestConstraints.java (original)
+++ openjpa/trunk/openjpa-integration/validation/src/test/java/org/apache/openjpa/integration/validation/TestConstraints.java Mon Apr  6 20:57:11 2015
@@ -1154,7 +1154,6 @@ public class TestConstraints extends Abs
     /**
      * Internal convenience method for getting the OpenJPA logger
      * 
-     * @return
      */
     private Log getLog() {
         return emf.getConfiguration().getLog("Tests");

Modified: openjpa/trunk/openjpa-integration/validation/src/test/java/org/apache/openjpa/integration/validation/TestMerge.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-integration/validation/src/test/java/org/apache/openjpa/integration/validation/TestMerge.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-integration/validation/src/test/java/org/apache/openjpa/integration/validation/TestMerge.java (original)
+++ openjpa/trunk/openjpa-integration/validation/src/test/java/org/apache/openjpa/integration/validation/TestMerge.java Mon Apr  6 20:57:11 2015
@@ -218,7 +218,6 @@ public class TestMerge extends AbstractP
     /**
      * Internal convenience method for getting the OpenJPA logger
      * 
-     * @return
      */
     private Log getLog() {
         return emf.getConfiguration().getLog("Tests");

Modified: openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/identifier/DBIdentifier.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/identifier/DBIdentifier.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/identifier/DBIdentifier.java (original)
+++ openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/identifier/DBIdentifier.java Mon Apr  6 20:57:11 2015
@@ -78,7 +78,6 @@ public class DBIdentifier extends Identi
 
     /**
      * Returns whether case is ignored during equality operations.
-     * @return
      */
     public boolean getIgnoreCase() {
         return _ignoreCase;
@@ -169,7 +168,6 @@ public class DBIdentifier extends Identi
      * @param name
      * @param id
      * @param delim
-     * @return
      */
     public static DBIdentifier[] split(String name, DBIdentifierType id, String delim) {
         
@@ -193,7 +191,6 @@ public class DBIdentifier extends Identi
      * returns a string based identifier.
      * @param resultId
      * @param names
-     * @return
      */
     public static String join(DBIdentifier...names) {
         if (names == null || names.length == 0) {
@@ -210,7 +207,6 @@ public class DBIdentifier extends Identi
      * type and heuristics to determine the types and placement of the resulting
      * components.
      * @param name
-     * @return
      */
     public static DBIdentifier[] split(DBIdentifierType resultType, String name) {
         
@@ -298,7 +294,6 @@ public class DBIdentifier extends Identi
     /**
      * Returns true if the identifier is null or the name is null or empty.
      * @param name
-     * @return
      */
     public static boolean isEmpty(DBIdentifier name) {
         if (isNull(name)) {
@@ -311,7 +306,6 @@ public class DBIdentifier extends Identi
     /**
      * Returns true if the identifier is null.
      * @param name
-     * @return
      */
     public static boolean isNull(DBIdentifier name) {
         return (name == null || name.getType() == DBIdentifierType.NULL);
@@ -349,7 +343,6 @@ public class DBIdentifier extends Identi
     /**
      * Returns a copy of an identifier with name trimmed to null.
      * @param name
-     * @return
      */
     public static DBIdentifier trimToNull(DBIdentifier name) {
         if (DBIdentifier.isNull(name)) {
@@ -382,7 +375,6 @@ public class DBIdentifier extends Identi
 
     /**
      * Returns true if this object is NULL or has a null name component.
-     * @return
      */
     public boolean isNull() {
         return (_type == DBIdentifierType.NULL || getName() == null);
@@ -410,7 +402,6 @@ public class DBIdentifier extends Identi
      * provided type
      * @param columnNames
      * @param id
-     * @return
      */
     public static DBIdentifier[] toArray(String[] columnNames, DBIdentifierType id) {
         return toArray(columnNames, id, false);
@@ -421,7 +412,6 @@ public class DBIdentifier extends Identi
      * provided type, optionally delimiting the names.
      * @param columnNames
      * @param id
-     * @return
      */
     public static DBIdentifier[] toArray(String[] names, DBIdentifierType id, boolean delimit) {
         if (names == null) {
@@ -437,7 +427,6 @@ public class DBIdentifier extends Identi
     /**
      * Returns a string array of names based upon the provided set of identifiers.
      * @param sNames
-     * @return
      */
     public static String[] toStringArray(DBIdentifier[] sNames) {
         if (sNames == null) {
@@ -645,7 +634,6 @@ public class DBIdentifier extends Identi
      * Static equality method for comparing two identifiers.
      * @param name1
      * @param name2
-     * @return
      */
     public static boolean equal(DBIdentifier name1, DBIdentifier name2) {
         boolean name1Null = DBIdentifier.isNull(name1);
@@ -669,7 +657,6 @@ public class DBIdentifier extends Identi
      * Returns a new DBIdentifier truncated to length
      * @param name
      * @param length
-     * @return
      */
     public static DBIdentifier truncate(DBIdentifier name, int length) {
         DBIdentifier sName = name.clone();
@@ -686,7 +673,6 @@ public class DBIdentifier extends Identi
      * Returns a new DBIdentifier with the given string appended.
      * @param name
      * @param length
-     * @return
      */
     public static DBIdentifier append(DBIdentifier name, String str) {
         DBIdentifier sName = name.clone();
@@ -701,7 +687,6 @@ public class DBIdentifier extends Identi
      * delimiting rules and appropriate separators.
      * @param name
      * @param length
-     * @return
      */
     public static DBIdentifier combine(DBIdentifier name, String str) {
         DBIdentifier sName = name.clone();
@@ -714,7 +699,6 @@ public class DBIdentifier extends Identi
     /**
      * Returns a new DBIdentifier converted to lower case - if not delimited.
      * @param name
-     * @return
      */
     public static DBIdentifier toLower(DBIdentifier name) {
         return toLower(name, false);
@@ -724,7 +708,6 @@ public class DBIdentifier extends Identi
      * Returns a new DBIdentifier converted to lower case.  If delimited,
      * force to lower case using force option.
      * @param name
-     * @return
      */
     public static DBIdentifier toLower(DBIdentifier name, boolean force) {
         if (DBIdentifier.isNull(name)) {
@@ -745,7 +728,6 @@ public class DBIdentifier extends Identi
     /**
      * Returns a new DBIdentifier converted to upper case - if not delimited.
      * @param name
-     * @return
      */
     public static DBIdentifier toUpper(DBIdentifier name) {
         return toUpper(name, false);
@@ -755,7 +737,6 @@ public class DBIdentifier extends Identi
      * Returns a new DBIdentifier converted to upper case.  If delimited,
      * force to upper case using force option.
      * @param name
-     * @return
      */
     public static DBIdentifier toUpper(DBIdentifier name, boolean force) {
         if (DBIdentifier.isNull(name)) {
@@ -776,7 +757,6 @@ public class DBIdentifier extends Identi
     /**
      * Returns a new DBIdentifier with the specified leading string removed.
      * @param name
-     * @return
      */
     public static DBIdentifier removeLeading(DBIdentifier name, String leadingStr) {
         DBIdentifier sName = name.clone();
@@ -795,7 +775,6 @@ public class DBIdentifier extends Identi
     /**
      * Returns a new DBIdentifier with Hungarian notation removed.
      * @param name
-     * @return
      */
     public static DBIdentifier removeHungarianNotation(DBIdentifier name) {
         DBIdentifier hName = name.clone();
@@ -811,7 +790,6 @@ public class DBIdentifier extends Identi
     /**
      * Equality operator which ignores case.
      * @param name
-     * @return
      */
    public boolean equalsIgnoreCase(DBIdentifier name) {
        return equals(name, true);
@@ -866,7 +844,6 @@ public class DBIdentifier extends Identi
     /**
      * Returns a new identifier with delimiters removed.
      * @param name
-     * @return
      */
     public static DBIdentifier removeDelimiters(DBIdentifier name) {
         if (DBIdentifier.isNull(name)) {
@@ -885,7 +862,6 @@ public class DBIdentifier extends Identi
     /**
      * Returns a new delimiter with leading and trailing spaces removed.
      * @param name
-     * @return
      */
     public static DBIdentifier trim(DBIdentifier name) {
         if (DBIdentifier.isNull(name)) {

Modified: openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/identifier/DBIdentifierUtil.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/identifier/DBIdentifierUtil.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/identifier/DBIdentifierUtil.java (original)
+++ openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/identifier/DBIdentifierUtil.java Mon Apr  6 20:57:11 2015
@@ -76,7 +76,6 @@ public interface DBIdentifierUtil extend
      * @param table
      * @param maxLen
      * @param checkForUniqueness
-     * @return
      */
     public DBIdentifier getValidColumnIdentifier(DBIdentifier name, Table table, int maxLen, 
         boolean checkForUniqueness);
@@ -88,7 +87,6 @@ public interface DBIdentifierUtil extend
      * @param table
      * @param maxLen
      * @param checkForUniqueness
-     * @return
      */
     public DBIdentifier getValidIndexIdentifier(DBIdentifier name, Table table, int maxLen);
 
@@ -99,7 +97,6 @@ public interface DBIdentifierUtil extend
      * @param table
      * @param maxLen
      * @param checkForUniqueness
-     * @return
      */
     public DBIdentifier getValidSequenceIdentifier(DBIdentifier name, Schema schema, int maxLen);
     
@@ -109,7 +106,6 @@ public interface DBIdentifierUtil extend
      * @param name
      * @param schema
      * @param maxLen
-     * @return
      */
     public DBIdentifier getValidTableIdentifier(DBIdentifier name, Schema schema, int maxLen);
 
@@ -119,7 +115,6 @@ public interface DBIdentifierUtil extend
      * @param name
      * @param table
      * @param maxLen
-     * @return
      */
     public DBIdentifier getValidUniqueIdentifier(DBIdentifier name, Table table, int maxLen);
 
@@ -130,14 +125,12 @@ public interface DBIdentifierUtil extend
      * @param table
      * @param toTable
      * @param maxLen
-     * @return
      */
     public DBIdentifier getValidForeignKeyIdentifier(DBIdentifier name, Table table, Table toTable, int maxLen);
 
     /**
      * Converts the specified identifier to a format required by the database.
      * @param name
-     * @return
      */
     public String toDBName(DBIdentifier name);
 
@@ -146,14 +139,12 @@ public interface DBIdentifierUtil extend
      * optionally delimiting the name.
      * @param name
      * @param delimit
-     * @return
      */
     public String toDBName(DBIdentifier name, boolean delimit);
 
     /**
      * Converts the specified string to a format required by the database.
      * @param name
-     * @return
      */
     public String toDBName(String name);
 
@@ -161,7 +152,6 @@ public interface DBIdentifierUtil extend
      * Converts the specified string to a format required by the database,
      * optionally delimiting the name.
      * @param name
-     * @return
      */
     public String toDBName(String name, boolean delimit);
 
@@ -169,14 +159,12 @@ public interface DBIdentifierUtil extend
      * Converts the name returned by the database to an identifier of the
      * specified type.
      * @param name
-     * @return
      */
     public DBIdentifier fromDBName(String name, DBIdentifierType id);
 
     /**
      * Appends multiple columns names together into comma delimited string.
      * @param columns
-     * @return
      */
     public String appendColumns(Column[] columns);
 
@@ -184,7 +172,6 @@ public interface DBIdentifierUtil extend
      * Converts the name of the specified delimiter to the appropriate
      * case as defined by the configuration.
      * @param columns
-     * @return
      */
     public DBIdentifier convertSchemaCase(DBIdentifier schema);
 
@@ -192,7 +179,6 @@ public interface DBIdentifierUtil extend
      * Appends multiple names together using the appropriate name delimiter.
      * @param resultId
      * @param names
-     * @return
      */
     public DBIdentifier append(DBIdentifierType resultId, DBIdentifier...names);
 
@@ -200,14 +186,12 @@ public interface DBIdentifierUtil extend
      * Returns a generated key sequence identifier for the column.
      * @param col
      * @param maxLen
-     * @return
      */
     public DBIdentifier getGeneratedKeySequenceName(Column col, int maxLen);
 
     /**
      * Converts a provided alias to a format specified in the configuration.
      * @param alias
-     * @return
      */
     public String convertAlias(String alias);
 }

Modified: openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/identifier/Normalizer.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/identifier/Normalizer.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/identifier/Normalizer.java (original)
+++ openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/identifier/Normalizer.java Mon Apr  6 20:57:11 2015
@@ -44,7 +44,6 @@ public class Normalizer {
     /**
      * Normalizes a multi-part name
      * @param name
-     * @return
      */
     public static String normalizeMulti(String name) {
         if (StringUtils.isEmpty(name)) {
@@ -57,7 +56,6 @@ public class Normalizer {
     /**
      * Normalizes a single part name
      * @param name
-     * @return
      */
     public static String normalizeString(String name) {
         if (StringUtils.isEmpty(name)) {
@@ -73,7 +71,6 @@ public class Normalizer {
     /**
      * Returns true if the name is delimited with default delimiters
      * @param name
-     * @return
      */
     public static boolean isDelimited(String name) {
         if (StringUtils.isEmpty(name)) {
@@ -87,7 +84,6 @@ public class Normalizer {
      * for equality
      * @param name1
      * @param name2
-     * @return
      */
     public static boolean fullNamesEqual(String name1, String name2) {
         if (StringUtils.isEmpty(name1) && StringUtils.isEmpty(name2)) {
@@ -120,7 +116,6 @@ public class Normalizer {
      * comparison.
      * @param name1
      * @param name2
-     * @return
      */
     public static boolean namesEqual(String name1, String name2) {
         if (StringUtils.isEmpty(name1) && StringUtils.isEmpty(name2)) {
@@ -141,7 +136,6 @@ public class Normalizer {
     /**
      * Normalizes a name, if not delimited, converts to uppercase.
      * @param name
-     * @return
      */
     public static String normalizeUpper(String name) {
         String nName = normalizeString(name);
@@ -156,7 +150,6 @@ public class Normalizer {
     /**
      * Splits a name into normalized components
      * @param name
-     * @return
      */
     public static String[] splitName(String name) {
         return normalizer.splitName(defaultRule, name);
@@ -167,7 +160,6 @@ public class Normalizer {
      * Splits a name into normalized components using the specified
      * name delimiter (ex. schema:table, delim = : --> { schema, table }
      * @param name
-     * @return
      */
     public static String[] splitName(String name, String delim) {
         return normalizer.splitName(defaultRule, name, delim);
@@ -176,7 +168,6 @@ public class Normalizer {
     /**
      * Joins multiple names using default identifier rules.
      * @param names
-     * @return
      */
     public static String joinNames(String[] names) {
         return normalizer.joinNames(defaultRule, names);
@@ -185,7 +176,6 @@ public class Normalizer {
     /**
      * Joins multiple names using the specified delimiter.
      * @param names
-     * @return
      */
     public static String joinNames(String[] names, String delimiter) {
         return normalizer.joinNames(defaultRule, names, delimiter);
@@ -194,7 +184,6 @@ public class Normalizer {
     /**
      * Joins two names using the default identifier rules.
      * @param names
-     * @return
      */
     public static String joinNames(String name1, String name2) {
         return joinNames(new String[] { name1, name2});
@@ -206,7 +195,6 @@ public class Normalizer {
      * delimiters.
      * @param name
      * @param length
-     * @return
      */
     public static String truncate(String name, int length) {
         return normalizer.truncateName(defaultRule, name, length);
@@ -219,7 +207,6 @@ public class Normalizer {
      * @param config
      * @param rule
      * @param name
-     * @return
      */
     public static String convert(IdentifierConfiguration config, String rule, String name) {
         return normalizer.convert(config, rule, name);
@@ -229,7 +216,6 @@ public class Normalizer {
      * Combines two names using default identifier rules.
      * @param name1
      * @param name2
-     * @return
      */
     public static String combine(String name1, String name2) {
         return normalizer.combineNames(defaultRule, name1, name2);
@@ -239,7 +225,6 @@ public class Normalizer {
      * Combines multiple names using default identifier rules.
      * @param name1
      * @param name2
-     * @return
      */
     public static String combine(String...names) {
         return normalizer.combineNames(defaultRule, names);
@@ -250,7 +235,6 @@ public class Normalizer {
      * Appends one string to another using default identifier rules.
      * @param name1
      * @param name2
-     * @return
      */
     public static String append(String name1, String name2) {
         return normalizer.appendNames(defaultRule, name1, name2);
@@ -260,7 +244,6 @@ public class Normalizer {
      * Removes Hungarian notation from a string.
      * @param name1
      * @param name2
-     * @return
      */
     public static String removeHungarianNotation(String name) {
         return normalizer.removeHungarianNotation(defaultRule, name);
@@ -270,7 +253,6 @@ public class Normalizer {
      * Removes default delimiters from a string.
      * @param name1
      * @param name2
-     * @return
      */
     public static String removeDelimiters(String name) {
         return normalizer.removeDelimiters(defaultRule, name);
@@ -281,7 +263,6 @@ public class Normalizer {
      * delimited.
      * @param name1
      * @param name2
-     * @return
      */
     public static String delimit(String name, boolean force) {
         return normalizer.delimit(defaultRule, name, force);
@@ -291,7 +272,6 @@ public class Normalizer {
      * Determines whether a name can be split into multiple components.
      * @param name1
      * @param name2
-     * @return
      */
     public static boolean canSplit(String name) {
         return normalizer.canSplit(defaultRule, name);
@@ -302,7 +282,6 @@ public class Normalizer {
      * into account the specified delimiter.
      * @param name1
      * @param name2
-     * @return
      */
     public static boolean canSplit(String name, String delim) {
         return normalizer.canSplit(defaultRule, name, delim);

Modified: openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/identifier/QualifiedDBIdentifier.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/identifier/QualifiedDBIdentifier.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/identifier/QualifiedDBIdentifier.java (original)
+++ openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/identifier/QualifiedDBIdentifier.java Mon Apr  6 20:57:11 2015
@@ -45,7 +45,6 @@ public class QualifiedDBIdentifier exten
      * Creates a qualified identifier based upon an array of DBIdentifiers.  Identifiers
      * must be specified in order.
      * @param names
-     * @return
      */
     public static QualifiedDBIdentifier newPath(DBIdentifier...names) {
         return new QualifiedDBIdentifier(names);
@@ -99,7 +98,6 @@ public class QualifiedDBIdentifier exten
     /**
      * Splits a qualified path into separate identifiers.
      * @param sName
-     * @return
      */
     public static DBIdentifier[] splitPath(DBIdentifier sName) {
         if (sName instanceof QualifiedDBIdentifier && sName.getType() != DBIdentifierType.SCHEMA) {
@@ -126,7 +124,6 @@ public class QualifiedDBIdentifier exten
     /**
      * Creates a qualified path from an identifier.
      * @param sName
-     * @return
      */
     public static QualifiedDBIdentifier getPath(DBIdentifier sName) {
         if (sName instanceof QualifiedDBIdentifier) {
@@ -144,7 +141,6 @@ public class QualifiedDBIdentifier exten
 
     /**
      * Gets the schema component of the path.
-     * @return
      */
     public DBIdentifier getSchemaName() {
         return _schemaName;
@@ -169,7 +165,6 @@ public class QualifiedDBIdentifier exten
 
     /**
      * Returns true if this object is not qualified by a schema component.
-     * @return
      */
     public boolean isUnqualifiedObject() {
         return DBIdentifier.isNull(getSchemaName());
@@ -178,7 +173,6 @@ public class QualifiedDBIdentifier exten
     /**
      * Returns true if this object is not qualified by a table or schema 
      * component.
-     * @return
      */
     public boolean isUnqualifiedColumn() {
         return DBIdentifier.isNull(getObjectTableName()) && DBIdentifier.isNull(getSchemaName());
@@ -211,7 +205,6 @@ public class QualifiedDBIdentifier exten
      * Compares two qualified identifiers for equality.
      * @param path1
      * @param path2
-     * @return
      */
     public static boolean pathEqual(QualifiedDBIdentifier path1, QualifiedDBIdentifier path2) {
         if (path1 == null && path2 == null) {
@@ -255,7 +248,6 @@ public class QualifiedDBIdentifier exten
     /**
      * Returns the base name of this qualified name.  For example, if the 
      * qualified name is a schema qualified table, the table name is returned.
-     * @return
      */
     public String getBaseName() {
         return super.getName();
@@ -263,7 +255,6 @@ public class QualifiedDBIdentifier exten
 
     /**
      * Returns this object as a DBIdentifier.
-     * @return
      */
     public DBIdentifier getIdentifier() {
         return this;

Modified: openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/meta/ReverseMappingTool.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/meta/ReverseMappingTool.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/meta/ReverseMappingTool.java (original)
+++ openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/meta/ReverseMappingTool.java Mon Apr  6 20:57:11 2015
@@ -2209,7 +2209,6 @@ public class ReverseMappingTool
         /**
          * Add the list of imports for any inner app id classes
          *
-         * @return
          */
         public Set getImportPackages() {
             Set pkgs = super.getImportPackages();

Modified: openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/meta/SequenceMapping.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/meta/SequenceMapping.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/meta/SequenceMapping.java (original)
+++ openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/meta/SequenceMapping.java Mon Apr  6 20:57:11 2015
@@ -269,7 +269,6 @@ public class SequenceMapping
 
     /**
      * @deprecated
-     * @return
      */
     public String getUniqueConstraintName() {
         return getUniqueConstraintIdentifier().getName();

Modified: openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/schema/SchemaGroup.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/schema/SchemaGroup.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/schema/SchemaGroup.java (original)
+++ openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/schema/SchemaGroup.java Mon Apr  6 20:57:11 2015
@@ -93,7 +93,6 @@ public class SchemaGroup
     /**
      * @deprecated
      * @param name
-     * @return
      */
     public Schema addSchema(String name) {
         return addSchema(DBIdentifier.newSchema(name));

Modified: openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/DBDictionary.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/DBDictionary.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/DBDictionary.java (original)
+++ openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/DBDictionary.java Mon Apr  6 20:57:11 2015
@@ -5071,7 +5071,6 @@ public class DBDictionary
      * Return version column name
      * @param column
      * @param tableAlias : this is needed for platform specific version column
-     * @return
      */
     public String getVersionColumn(Column column, String tableAlias) {
         return getVersionColumn(column, DBIdentifier.newTable(tableAlias)).toString();
@@ -5580,7 +5579,6 @@ public class DBDictionary
 
     /**
      * Returns the naming utility used by this dictionary instance
-     * @return
      */
     public DBIdentifierUtil getNamingUtil() {
         return namingUtil;

Modified: openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/Joins.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/Joins.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/Joins.java (original)
+++ openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/Joins.java Mon Apr  6 20:57:11 2015
@@ -94,7 +94,6 @@ public interface Joins {
 
     /**
      * Return correlated variable name
-     * @return
      */
     public String getCorrelatedVariable();
 

Modified: openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/PostgresDictionary.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/PostgresDictionary.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/PostgresDictionary.java (original)
+++ openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/PostgresDictionary.java Mon Apr  6 20:57:11 2015
@@ -547,7 +547,6 @@ public class PostgresDictionary
      * db.  This logic assumes that any sequence suffixed with _SEQ is an
      * owned sequence (identical to the behavior of prior versions of OpenJPA).
      * @param strName
-     * @return
      */
     private boolean isOwnedSequence(String strName) {
         // filter out generated sequences used for bigserial cols, which are
@@ -786,7 +785,6 @@ public class PostgresDictionary
      * @param column
      * @param map
      * 
-     * @return
      * @exception SQLException
      */
     public Object getObject(ResultSet rs, int column, Map map)

Modified: openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/Select.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/Select.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/Select.java (original)
+++ openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/Select.java Mon Apr  6 20:57:11 2015
@@ -729,7 +729,6 @@ public interface Select
     /**
      * Return the flag to indicate whether this Select has
      * internally generated subselect
-     * @return
      */
     public boolean getHasSubselect();
    

Modified: openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/SelectImpl.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/SelectImpl.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/SelectImpl.java (original)
+++ openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/SelectImpl.java Mon Apr  6 20:57:11 2015
@@ -3376,7 +3376,6 @@ interface PathJoins
 
     /**
      * The select owner of this join
-     * @return
      */
     public Select getSelect();
     

Modified: openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/json/JSONObject.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/json/JSONObject.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/json/JSONObject.java (original)
+++ openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/json/JSONObject.java Mon Apr  6 20:57:11 2015
@@ -83,7 +83,6 @@ public class JSONObject implements JSON
      * @param field
      * @param value
      * @param indent
-     * @return
      */
     private static StringBuilder encodeField(String field, Object value, int indent) {
         return new StringBuilder()
@@ -114,7 +113,6 @@ public class JSONObject implements JSON
      * If the value is JSON, outputs the string rendition of value
      * Otherwise quoted value
      * @param o
-     * @return
      */
     private static StringBuilder quoteFieldValue(Object o, int indent) {
         if (o == null) return new StringBuilder(NULL_LITERAL);

Modified: openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/IOR.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/IOR.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/IOR.java (original)
+++ openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/IOR.java Mon Apr  6 20:57:11 2015
@@ -36,7 +36,6 @@ public class IOR {
      * The simple Java type name and the persistent identity separated by a {@link Constants#DASH dash}.
      *  
      * @param sm a managed instance.
-     * @return
      */
     public static String toString(OpenJPAStateManager sm) {
         if (sm == null) return NULL_VALUE;

Modified: openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/JESTContext.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/JESTContext.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/JESTContext.java (original)
+++ openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/JESTContext.java Mon Apr  6 20:57:11 2015
@@ -275,7 +275,6 @@ public class JESTContext implements JPAS
     /**
      * Is this path a context root?
      * @param path
-     * @return
      */
     boolean isContextRoot(String path) {
         return (path == null || CONTEXT_ROOT.equals(path));

Modified: openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/JPAServletContext.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/JPAServletContext.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/JPAServletContext.java (original)
+++ openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/JPAServletContext.java Mon Apr  6 20:57:11 2015
@@ -56,7 +56,6 @@ public interface JPAServletContext {
     
     /**
      * Get the requested URI. 
-     * @return
      */
     public String getRequestURI();
     

Modified: openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/JSONObject.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/JSONObject.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/JSONObject.java (original)
+++ openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/JSONObject.java Mon Apr  6 20:57:11 2015
@@ -83,7 +83,6 @@ public class JSONObject implements JSON
      * @param field
      * @param value
      * @param indent
-     * @return
      */
     private static StringBuilder encodeField(String field, Object value, int indent) {
         return new StringBuilder()
@@ -114,7 +113,6 @@ public class JSONObject implements JSON
      * If the value is JSON, outputs the string rendition of value
      * Otherwise quoted value
      * @param o
-     * @return
      */
     private static StringBuilder quoteFieldValue(Object o, int indent) {
         if (o == null) return new StringBuilder(NULL_LITERAL);

Modified: openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/JSONObjectFormatter.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/JSONObjectFormatter.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/JSONObjectFormatter.java (original)
+++ openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/JSONObjectFormatter.java Mon Apr  6 20:57:11 2015
@@ -262,7 +262,6 @@ public class JSONObjectFormatter impleme
      * to be included in CDATA section of a XML document.
      * 
      * @param value the field value to be converted. Can not be null 
-     * @return
      */
     String streamToString(Object value) {
         Reader reader = null;

Modified: openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/MetamodelHelper.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/MetamodelHelper.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/MetamodelHelper.java (original)
+++ openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/MetamodelHelper.java Mon Apr  6 20:57:11 2015
@@ -68,7 +68,6 @@ public class MetamodelHelper {
     /**
      * Gets the attributes of the given type in defined order.
      * @param type
-     * @return
      */
     public static List<Attribute<?,?>> getAttributesInOrder(ManagedType<?> type) {
         List<Attribute<?,?>> list = new ArrayList<Attribute<?,?>>(type.getAttributes());
@@ -108,7 +107,6 @@ public class MetamodelHelper {
      * Gets a ordinal value of enumerated persistent attribute category.
      *  
      * @param attr
-     * @return
      */
     public static AttributeCategory getAttributeCategory(Attribute<?,?> attr) {
         if (isId(attr))
@@ -145,7 +143,6 @@ public class MetamodelHelper {
      * Gets name of the attribute type. For collection and map type attribute, the name is
      * appended with generic type argument names.
      * @param attr
-     * @return
      */
     public static String getAttributeTypeName(Attribute<?, ?> attr) {
         StringBuilder name = new StringBuilder(attr.getJavaType().getSimpleName());

Modified: openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/TokenReplacedStream.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/TokenReplacedStream.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/TokenReplacedStream.java (original)
+++ openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/TokenReplacedStream.java Mon Apr  6 20:57:11 2015
@@ -105,7 +105,6 @@ public class TokenReplacedStream {
     /**
      * Gets the pattern if any in matched state
      * @param patterns
-     * @return
      */
     Pattern matched(Pattern...patterns) {
         if (patterns == null)

Modified: openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/XMLFormatter.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/XMLFormatter.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/XMLFormatter.java (original)
+++ openjpa/trunk/openjpa-jest/src/main/java/org/apache/openjpa/persistence/jest/XMLFormatter.java Mon Apr  6 20:57:11 2015
@@ -453,7 +453,6 @@ public class XMLFormatter implements Obj
      * to be included in CDATA section of a XML document.
      * 
      * @param value the field value to be converted. Can not be null 
-     * @return
      */
     private String streamToString(Object value) {
         Reader reader = null;

Modified: openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/conf/Specification.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/conf/Specification.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/conf/Specification.java (original)
+++ openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/conf/Specification.java Mon Apr  6 20:57:11 2015
@@ -153,7 +153,6 @@ public class Specification {
     
     /**
      * Return the compatibility object associated with this Specification instance.
-     * @return
      */
     public Compatibility getCompatibility() {
         return _compatibility;

Modified: openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/instrumentation/AbstractQueryCacheInstrument.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/instrumentation/AbstractQueryCacheInstrument.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/instrumentation/AbstractQueryCacheInstrument.java (original)
+++ openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/instrumentation/AbstractQueryCacheInstrument.java Mon Apr  6 20:57:11 2015
@@ -182,7 +182,6 @@ public abstract class AbstractQueryCache
 
     /**
      * Returns all query keys currently tracked in the cache.
-     * @return
      */
     public Set<String> queryKeys() {
         QueryStatistics<QueryKey> stats = getStatistics();

Modified: openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/instrumentation/PreparedQueryCacheInstrument.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/instrumentation/PreparedQueryCacheInstrument.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/instrumentation/PreparedQueryCacheInstrument.java (original)
+++ openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/instrumentation/PreparedQueryCacheInstrument.java Mon Apr  6 20:57:11 2015
@@ -82,7 +82,6 @@ public interface PreparedQueryCacheInstr
     
     /**
      * Returns all queries currently tracked in the cache.
-     * @return
      */
     public Set<String> queries();
 }

Modified: openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/instrumentation/QueryCacheInstrument.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/instrumentation/QueryCacheInstrument.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/instrumentation/QueryCacheInstrument.java (original)
+++ openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/instrumentation/QueryCacheInstrument.java Mon Apr  6 20:57:11 2015
@@ -88,7 +88,6 @@ public interface QueryCacheInstrument {
     
     /**
      * Returns all the string-ified keys for query results in the cache.
-     * @return
      */
     public Set<String> queryKeys();
 

Modified: openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/instrumentation/jmx/JMXInstrument.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/instrumentation/jmx/JMXInstrument.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/instrumentation/jmx/JMXInstrument.java (original)
+++ openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/instrumentation/jmx/JMXInstrument.java Mon Apr  6 20:57:11 2015
@@ -29,7 +29,6 @@ public interface JMXInstrument extends I
     
     /**
      * Returns the JMX object name for the instrument
-     * @return
      */
     public ObjectName getObjectName();
 

Modified: openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/instrumentation/jmx/JMXProvider.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/instrumentation/jmx/JMXProvider.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/instrumentation/jmx/JMXProvider.java (original)
+++ openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/instrumentation/jmx/JMXProvider.java Mon Apr  6 20:57:11 2015
@@ -69,7 +69,6 @@ public class JMXProvider
 
     /**
      * Returns the mbean server
-     * @return
      */
     public Set<MBeanServer> getMBeanServer() {
         if (_mbs == null) {

Modified: openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/FinderQuery.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/FinderQuery.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/FinderQuery.java (original)
+++ openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/FinderQuery.java Mon Apr  6 20:57:11 2015
@@ -39,7 +39,6 @@ public interface FinderQuery<K,V,R>  {
     /**
      * Gets the value to which this receiver delegates its execution.
      * 
-     * @return
      */
     public V getDelegate();
     

Modified: openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/exps/AbstractExpressionBuilder.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/exps/AbstractExpressionBuilder.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/exps/AbstractExpressionBuilder.java (original)
+++ openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/exps/AbstractExpressionBuilder.java Mon Apr  6 20:57:11 2015
@@ -610,7 +610,6 @@ public abstract class AbstractExpression
     /**
      * Returns the variable associated with the schema alias (id).
      * @param id
-     * @return
      */
     protected abstract Value getVariable(String id);
 }

Modified: openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/exps/Context.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/exps/Context.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/exps/Context.java (original)
+++ openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/exps/Context.java Mon Apr  6 20:57:11 2015
@@ -78,7 +78,6 @@ public class Context implements Serializ
 
     /**
      * Returns next table alias to be created.
-     * @return
      */
     public int nextAlias() {
         Context p = this;
@@ -111,7 +110,6 @@ public class Context implements Serializ
 
     /**
      * Returns the select associated with this context.
-     * @return
      */
     public Object getSelect() {
         return select;
@@ -131,7 +129,6 @@ public class Context implements Serializ
 
     /**
      * Returns the subquery context.
-     * @return
      */
     public List<Context> getSubselContexts() {
         return subsels;
@@ -139,7 +136,6 @@ public class Context implements Serializ
 
     /**
      * Returns the subquery in this context.
-     * @return
      */
     public Subquery getSubquery() {
         return subquery;
@@ -182,7 +178,6 @@ public class Context implements Serializ
     /**
      * Given an alias and return its associated variable.
      * @param alias
-     * @return
      */
     public Value getVariable(String alias) {
         Value variable = alias == null ? null 
@@ -194,7 +189,6 @@ public class Context implements Serializ
      * Given an alias find the context of its associated
      * variable where it is defined.
      * @param alias
-     * @return
      */
     public Context findContext(String alias) {
         Value var = getVariable(alias);
@@ -221,7 +215,6 @@ public class Context implements Serializ
     /**
      * Given an alias find the variable in JPQL contexts.
      * @param alias
-     * @return
      */
     public Value findVariable(String alias) {
         Value var = getVariable(alias);

Modified: openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/exps/QueryExpressions.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/exps/QueryExpressions.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/exps/QueryExpressions.java (original)
+++ openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/exps/QueryExpressions.java Mon Apr  6 20:57:11 2015
@@ -90,7 +90,6 @@ public class QueryExpressions
 
     /**
      * Returns the current JPQL query context.
-     * @return
      */
     public Context ctx() {
         return _contexts.peek();

Modified: openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/meta/ClassMetaData.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/meta/ClassMetaData.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/meta/ClassMetaData.java (original)
+++ openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/meta/ClassMetaData.java Mon Apr  6 20:57:11 2015
@@ -2686,7 +2686,6 @@ public class ClassMetaData
      * method is used by the PCEnhancer to determine if special handling is
      * required.
      *
-     * @return
      */
     public boolean hasAbstractPKField() {
         if (_hasAbstractPKField != null) {
@@ -2722,7 +2721,6 @@ public class ClassMetaData
      * between the type represented by this ClassMetaData object and the
      * abstract pcType declaring PKFields.
      *
-     * @return
      */
     public boolean hasPKFieldsFromAbstractClass() {
         if (_hasPKFieldsFromAbstractClass != null) {

Modified: openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/meta/MetaDataDefaults.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/meta/MetaDataDefaults.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/meta/MetaDataDefaults.java (original)
+++ openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/meta/MetaDataDefaults.java Mon Apr  6 20:57:11 2015
@@ -129,7 +129,6 @@ public interface MetaDataDefaults
     
     /**
      * Whether cascade-persist was declared in the persistence unit defaults.
-     * @return
      */
     public Boolean isDefaultCascadePersistEnabled();
     

Modified: openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/meta/ValueMetaDataImpl.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/meta/ValueMetaDataImpl.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/meta/ValueMetaDataImpl.java (original)
+++ openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/meta/ValueMetaDataImpl.java Mon Apr  6 20:57:11 2015
@@ -249,7 +249,6 @@ public class ValueMetaDataImpl
     /**
      * Check if the persistence unit default <cascade-persist> has been enabled.  If so, then change
      * CASCADE_NONE to CASCADE_IMMEDIATE.
-     * @return
      */
     private int checkPUDefaultCascadePersist() {
         if (_checkPUDefaultCascadePersist) {

Modified: openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/util/DelayedProxy.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/util/DelayedProxy.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/util/DelayedProxy.java (original)
+++ openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/util/DelayedProxy.java Mon Apr  6 20:57:11 2015
@@ -72,13 +72,11 @@ public interface DelayedProxy {
     /**
      * Returns the expected field index even if this collection
      * was detached.
-     * @return
      */
     int getDelayedField();
     
     /**
      * Returns whether the proxy is detached.
-     * @return
      */
     boolean isDetached();
 }

Modified: openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/ConfigurationImpl.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/ConfigurationImpl.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/ConfigurationImpl.java (original)
+++ openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/ConfigurationImpl.java Mon Apr  6 20:57:11 2015
@@ -601,7 +601,6 @@ public class ConfigurationImpl
      * @param storeDefaults
      *            whether or not to retrieve a property if its value is the
      *            default value.
-     * @return
      */
     public Map toProperties(boolean storeDefaults) {
         // clone properties before making any modifications; we need to keep

Modified: openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/Configurations.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/Configurations.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/Configurations.java (original)
+++ openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/Configurations.java Mon Apr  6 20:57:11 2015
@@ -191,7 +191,6 @@ public class Configurations {
      * For efficiency, a cache per class loader is maintained of classes already loader. 
      * @param clsName
      * @param loader
-     * @return
      */
     static Class<?> loadClass(String clsName, ClassLoader loader) {
         Class<?> cls = null; 

Modified: openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/identifier/IdentifierConfiguration.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/identifier/IdentifierConfiguration.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/identifier/IdentifierConfiguration.java (original)
+++ openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/identifier/IdentifierConfiguration.java Mon Apr  6 20:57:11 2015
@@ -65,7 +65,6 @@ public interface IdentifierConfiguration
     
     /**
      * Returns all naming rules 
-     * @return
      */
     public <T> Map<T, IdentifierRule> getIdentifierRules();
     
@@ -92,7 +91,6 @@ public interface IdentifierConfiguration
      * should take place.  Id configurations should create a key unique
      * to their configuration.  The typical key is:
      * leading delimiter (") + name separator(.) + trailing delimiter(")
-     * @return
      */
     public String getConversionKey();
 }

Modified: openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/identifier/IdentifierUtilImpl.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/identifier/IdentifierUtilImpl.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/identifier/IdentifierUtilImpl.java (original)
+++ openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/identifier/IdentifierUtilImpl.java Mon Apr  6 20:57:11 2015
@@ -187,7 +187,6 @@ public class IdentifierUtilImpl implemen
      * Join names using a single naming rule and specified delimiter
      * @param rule
      * @param names
-     * @return
      */
     public String joinNames(IdentifierConfiguration config, IdentifierRule rule, String[] names, String delimiter) {
         
@@ -250,7 +249,6 @@ public class IdentifierUtilImpl implemen
      * Gets the first part of a name when single character delimiters are
      * in use.
      * @param pname
-     * @return
      */
     private String splitNameCharDelimiters(String name, String nameDelim) {
         StringBuilder sname = new StringBuilder("");
@@ -284,7 +282,6 @@ public class IdentifierUtilImpl implemen
 
     /**
      * Returns whether a name is double quoted
-     * @return
      */
     public static boolean isDoubleQuoted(String name) {
         if (name == null || name.length() < 3) {

Modified: openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/instrumentation/Instrument.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/instrumentation/Instrument.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/instrumentation/Instrument.java (original)
+++ openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/instrumentation/Instrument.java Mon Apr  6 20:57:11 2015
@@ -82,7 +82,6 @@ public interface Instrument {
 
     /**
      * Returns true if the instrument is started.
-     * @return
      */
     public boolean isStarted();
 

Modified: openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/instrumentation/InstrumentationProvider.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/instrumentation/InstrumentationProvider.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/instrumentation/InstrumentationProvider.java (original)
+++ openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/instrumentation/InstrumentationProvider.java Mon Apr  6 20:57:11 2015
@@ -66,7 +66,6 @@ public interface InstrumentationProvider
     /**
      * Returns the string-based list of instruments directly configured by
      * this provider via setInstrument.
-     * @return
      */
     public String getInstrument();
  

Modified: openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/callbacks/PostPersistEntity.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/callbacks/PostPersistEntity.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/callbacks/PostPersistEntity.java (original)
+++ openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/callbacks/PostPersistEntity.java Mon Apr  6 20:57:11 2015
@@ -63,7 +63,6 @@ public class PostPersistEntity {
     /**
      * Gets value of the named field by reflection to ensure that the field is
      * not loaded as a side-effect to access it.
-     * @return
      */
     Object getByReflection(String f) {
         try {

Modified: openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/common/utils/AbstractTestCase.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/common/utils/AbstractTestCase.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/common/utils/AbstractTestCase.java (original)
+++ openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/common/utils/AbstractTestCase.java Mon Apr  6 20:57:11 2015
@@ -210,7 +210,6 @@ public abstract class AbstractTestCase e
      * Creates a EMF and adds it to a Map for cleanup in tearDown()
      * 
      * @param map
-     * @return
      */
     protected OpenJPAEntityManagerFactory getEmf(Map map) {
         if (map == null)
@@ -259,7 +258,6 @@ public abstract class AbstractTestCase e
     /**
      * Creates a EMF and adds it to a Map for cleanup in tearDown()
      * 
-     * @return
      */
     protected OpenJPAEntityManagerFactory getEmf() {
         Map m = new HashMap();

Modified: openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/datacache/TestDataCacheBehavesIdentical.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/datacache/TestDataCacheBehavesIdentical.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/datacache/TestDataCacheBehavesIdentical.java (original)
+++ openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/datacache/TestDataCacheBehavesIdentical.java Mon Apr  6 20:57:11 2015
@@ -373,7 +373,6 @@ public class TestDataCacheBehavesIdentic
      * @param useDataCache
      * @param lock
      * @param makeDirtyBeforeRefresh
-     * @return
      */
     String getExpectedMarker(boolean useDataCache, LockModeType lock, 
             boolean makeDirtyBeforeRefresh) {

Modified: openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/graph/Graph.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/graph/Graph.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/graph/Graph.java (original)
+++ openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/graph/Graph.java Mon Apr  6 20:57:11 2015
@@ -95,7 +95,6 @@ public interface Graph<E> extends Set<E>
      * Gets all the relations originating from the given source.
      * @param <V>
      * @param source
-     * @return
      */
     public <V extends E> Set<Relation<V,E>> getRelationsFrom(V source);
     
@@ -103,7 +102,6 @@ public interface Graph<E> extends Set<E>
      * Gets all the relations terminating on the given target.
      * @param <V>
      * @param target
-     * @return
      */
     public <V extends E> Set<Relation<E,V>> getRelationsTo(V target);
 }

Modified: openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/kernel/TestFetchPlan.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/kernel/TestFetchPlan.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/kernel/TestFetchPlan.java (original)
+++ openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/kernel/TestFetchPlan.java Mon Apr  6 20:57:11 2015
@@ -263,7 +263,6 @@ public class TestFetchPlan extends BaseK
      *
      * @param dir a starting directory
      * @param depth depth to recurse. must be positive.
-     * @return
      */
     PCDirectory[] getParents(PCDirectory dir, int depth) {
         PCDirectory[] result = new PCDirectory[depth];

Modified: openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/query/TestMethodQLQuery.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/query/TestMethodQLQuery.java?rev=1671694&r1=1671693&r2=1671694&view=diff
==============================================================================
--- openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/query/TestMethodQLQuery.java (original)
+++ openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/query/TestMethodQLQuery.java Mon Apr  6 20:57:11 2015
@@ -94,7 +94,6 @@ public class TestMethodQLQuery
     
     /**
      * Returns the list whose element is the Map of input parameters.
-     * @return
      */
     public static ResultObjectProvider echo(StoreContext ctx,
         ClassMetaData meta, boolean subs, Map params, FetchConfiguration conf) {