You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@uima.apache.org by re...@apache.org on 2020/09/15 21:07:44 UTC

[uima-uimaj] branch UIMA-3484-Fix-JavaDoc created (now bbec5c4)

This is an automated email from the ASF dual-hosted git repository.

rec pushed a change to branch UIMA-3484-Fix-JavaDoc
in repository https://gitbox.apache.org/repos/asf/uima-uimaj.git.


      at bbec5c4  [UIMA-3484] Fix JavaDoc issues

This branch includes the following new commits:

     new bbec5c4  [UIMA-3484] Fix JavaDoc issues

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.



[uima-uimaj] 01/01: [UIMA-3484] Fix JavaDoc issues

Posted by re...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

rec pushed a commit to branch UIMA-3484-Fix-JavaDoc
in repository https://gitbox.apache.org/repos/asf/uima-uimaj.git

commit bbec5c48ebdae4f69ac386fa0d88aa2653bb0a17
Author: Richard Eckart de Castilho <re...@apache.org>
AuthorDate: Tue Sep 15 23:07:41 2020 +0200

    [UIMA-3484] Fix JavaDoc issues
    
    - First bunch of JavaDoc warnings fixed
    - Adds the "public abstract FloatArray getFloatArray0L();" method to the JCas interface which was previously not present due to a missing JavaDoc closing marker
---
 .../org/apache/vinci/transport/BaseClient.java     |  4 +-
 .../org/apache/vinci/transport/VinciServer.java    |  1 -
 .../vinci/VinciAnalysisEngineServiceStub.java      | 27 ++++---
 .../vinci/VinciAnalysisEngineService_impl.java     | 11 ++-
 .../src/main/java/org/apache/uima/cas/FSIndex.java |  3 +-
 .../src/main/java/org/apache/uima/jcas/JCas.java   |  4 +
 .../collection/impl/cpm/engine/ProcessingUnit.java |  7 +-
 .../collection/impl/cpm/utils/ChunkMetadata.java   |  3 -
 .../collection/impl/cpm/vinci/DATACasUtils.java    | 67 ----------------
 .../uima/collection/impl/cpm/vinci/Vinci.java      | 13 ++--
 .../impl/metadata/cpe/CasProcessorCpeObject.java   | 52 ++++++++++---
 .../impl/metadata/cpe/CpeCasProcessorsImpl.java    | 23 ++++++
 .../impl/metadata/cpe/CpeCheckpointImpl.java       | 28 ++++---
 .../cpe/CpeCollectionReaderCasInitializerImpl.java | 17 ++---
 .../uima/caseditor/ide/CasEditorIdePlugin.java     | 11 +--
 .../uima/taeconfigurator/editors/ui/Utility.java   | 88 ++++++++++++----------
 .../apache/uima/ep_launcher/LauncherPlugin.java    | 19 +++--
 .../uima/pear/insd/edit/INSDComponentPage.java     | 61 +++++++++------
 .../edit/vars/AbstractVarValViewerHandler.java     | 33 ++++----
 .../uima/pear/insd/edit/vars/VarValList.java       | 42 +++++------
 .../org/apache/uima/examples/cpe/SimpleRunCPE.java | 21 ++++--
 .../uima/examples/xmi/Ecore2UimaTypeSystem.java    | 16 ++--
 .../apache/uima/tools/AnnotationViewerMain.java    | 20 +++--
 .../src/main/java/org/apache/uima/tools/RunAE.java | 22 ++++--
 .../java/org/apache/uima/tools/cvd/MainFrame.java  | 10 ++-
 .../uima/tools/cvd/MultiAnnotViewerFrame.java      | 14 ----
 .../uima/tools/cvd/control/IndexPopupListener.java |  5 +-
 .../tools/cvd/control/ShowTypesystemHandler.java   |  4 +-
 .../uima/tools/cvd/tsview/FeatureTableModel.java   | 13 +---
 .../apache/uima/tools/cvd/tsview/MainFrame.java    | 18 +----
 .../uima/tools/docanalyzer/DocumentAnalyzer.java   | 73 ++++++++++++------
 .../apache/uima/tools/stylemap/StyleMapEditor.java | 66 ++++++++++------
 .../uima/tools/util/gui/SpringUtilities.java       |  1 +
 33 files changed, 431 insertions(+), 366 deletions(-)

diff --git a/jVinci/src/main/java/org/apache/vinci/transport/BaseClient.java b/jVinci/src/main/java/org/apache/vinci/transport/BaseClient.java
index 591cb5f..0a078e7 100644
--- a/jVinci/src/main/java/org/apache/vinci/transport/BaseClient.java
+++ b/jVinci/src/main/java/org/apache/vinci/transport/BaseClient.java
@@ -415,12 +415,10 @@ public class BaseClient {
    * case where return result is known to be VinciFrame (eliminates the need for casting in the
    * typical usage case).
    * 
-   * @return A VinciFrame representing the service result.
-   * 
    * @pre query != null
    * @pre getHost() != null
    * @param query -
-   * @return -
+   * @return A VinciFrame representing the service result.
    * @throws IOException -
    * @throws ServiceException -
    */
diff --git a/jVinci/src/main/java/org/apache/vinci/transport/VinciServer.java b/jVinci/src/main/java/org/apache/vinci/transport/VinciServer.java
index 2f36272..10c3c78 100644
--- a/jVinci/src/main/java/org/apache/vinci/transport/VinciServer.java
+++ b/jVinci/src/main/java/org/apache/vinci/transport/VinciServer.java
@@ -268,7 +268,6 @@ public class VinciServer extends BaseServer {
    * @param shutdown_message
    *          Should be used to pass a message explaining the shutdown, or in the future it may also
    *          include authentication information for password-protected shutdown.
-   * @param shutdown_message -
    * @return -
    */
   public static Transportable createShutdownCommand(String shutdown_message) {
diff --git a/uimaj-adapter-vinci/src/main/java/org/apache/uima/adapter/vinci/VinciAnalysisEngineServiceStub.java b/uimaj-adapter-vinci/src/main/java/org/apache/uima/adapter/vinci/VinciAnalysisEngineServiceStub.java
index ac2c91a..20c49d5 100644
--- a/uimaj-adapter-vinci/src/main/java/org/apache/uima/adapter/vinci/VinciAnalysisEngineServiceStub.java
+++ b/uimaj-adapter-vinci/src/main/java/org/apache/uima/adapter/vinci/VinciAnalysisEngineServiceStub.java
@@ -98,13 +98,15 @@ public class VinciAnalysisEngineServiceStub implements AnalysisEngineServiceStub
       }
       if (vnsHost == null) {
         vnsHost = System.getProperty("VNS_HOST");
-        if (vnsHost == null)
-          vnsHost = Constants.DEFAULT_VNS_HOST;
+        if (vnsHost == null) {
+            vnsHost = Constants.DEFAULT_VNS_HOST;
+        }
       }
       if (vnsPort == null) {
         vnsPort = System.getProperty("VNS_PORT");
-        if (vnsPort == null)
-          vnsPort = "9000";
+        if (vnsPort == null) {
+            vnsPort = "9000";
+        }
       }
       vctx.setVNSHost(vnsHost);
       vctx.setVNSPort(Integer.parseInt(vnsPort));
@@ -145,7 +147,8 @@ public class VinciAnalysisEngineServiceStub implements AnalysisEngineServiceStub
    * @see ResourceServiceStub#callGetMetaData()
    * @return -
    */
-  public ResourceMetaData callGetMetaData() throws ResourceServiceException {
+  @Override
+public ResourceMetaData callGetMetaData() throws ResourceServiceException {
     try {
       // create Vinci Frame
       VinciFrame queryFrame = new VinciFrame();
@@ -197,11 +200,13 @@ public class VinciAnalysisEngineServiceStub implements AnalysisEngineServiceStub
    * @param aCAS theCAS
    * @throws ResourceServiceException the resource service exception
    */
-  public void callProcess(CAS aCAS) throws ResourceServiceException {
+  @Override
+public void callProcess(CAS aCAS) throws ResourceServiceException {
     doProcess(aCAS);
   }
 
   /**
+   * @param aCAS -
    * @see CasObjectProcessor#processCas(CAS)
    * @throws ResourceServiceException -
    */
@@ -221,6 +226,7 @@ public class VinciAnalysisEngineServiceStub implements AnalysisEngineServiceStub
       query.setCommand("Annotate");
 
       mVinciClient.sendAndReceive(query, new TransportableFactory() {
+        @Override
         public Transportable makeTransportable() {
           // query.ignoreResponse = true; // TESTING
           return query;
@@ -243,14 +249,16 @@ public class VinciAnalysisEngineServiceStub implements AnalysisEngineServiceStub
   /**
    * @see ResourceServiceStub#destroy()
    */
-  public void destroy() {
+  @Override
+public void destroy() {
     mVinciClient.close();
   }
 
   /**
    * @see CasObjectProcessor#batchProcessComplete(org.apache.uima.util.ProcessTrace)
    */
-  public void callBatchProcessComplete() throws ResourceServiceException {
+  @Override
+public void callBatchProcessComplete() throws ResourceServiceException {
     try {
       // create Vinci Frame ( Data Cargo)
       AFrame queryFrame = new AFrame();
@@ -266,7 +274,8 @@ public class VinciAnalysisEngineServiceStub implements AnalysisEngineServiceStub
   /**
    * @see CasObjectProcessor#collectionProcessComplete(org.apache.uima.util.ProcessTrace)
    */
-  public void callCollectionProcessComplete() throws ResourceServiceException {
+  @Override
+public void callCollectionProcessComplete() throws ResourceServiceException {
     try {
       // create Vinci Frame ( Data Cargo)
       AFrame queryFrame = new AFrame();
diff --git a/uimaj-adapter-vinci/src/main/java/org/apache/uima/adapter/vinci/VinciAnalysisEngineService_impl.java b/uimaj-adapter-vinci/src/main/java/org/apache/uima/adapter/vinci/VinciAnalysisEngineService_impl.java
index 2a197a7..fb81e09 100644
--- a/uimaj-adapter-vinci/src/main/java/org/apache/uima/adapter/vinci/VinciAnalysisEngineService_impl.java
+++ b/uimaj-adapter-vinci/src/main/java/org/apache/uima/adapter/vinci/VinciAnalysisEngineService_impl.java
@@ -132,6 +132,7 @@ public class VinciAnalysisEngineService_impl extends VinciServableAdapter {
    * Instantiate Analysis Engine service from a given descriptor.
    * 
    * @param serviceConfigPath descriptor location
+   * @throws Exception -
    */
   public VinciAnalysisEngineService_impl(String serviceConfigPath) throws Exception {
     this(serviceConfigPath, false);
@@ -141,8 +142,7 @@ public class VinciAnalysisEngineService_impl extends VinciServableAdapter {
    * Extracts AE metadata
    * 
    * @return Frame containing extracted meta data
-   * @exception when
-   *              there is a failure processing
+   * @throws Exception when there is a failure processing
    */
   private Frame getMetaData() throws Exception {
     UIMAFramework.getLogger().log(Level.FINEST, "VinciAnalysisEngineService.getMetaData()");
@@ -216,7 +216,8 @@ public class VinciAnalysisEngineService_impl extends VinciServableAdapter {
    *         performing the service
    */
 
-  public Transportable eval(Transportable doc) throws ServiceException {
+  @Override
+public Transportable eval(Transportable doc) throws ServiceException {
     try {
       CASTransportable ct = (CASTransportable) doc;
       String op = ct.getCommand();
@@ -396,6 +397,7 @@ public class VinciAnalysisEngineService_impl extends VinciServableAdapter {
       server = instance;
     }
 
+    @Override
     public void run() {
       server.stop();
     }
@@ -404,7 +406,8 @@ public class VinciAnalysisEngineService_impl extends VinciServableAdapter {
   /**
    * @see org.apache.vinci.transport.TransportableFactory#makeTransportable()
    */
-  public synchronized Transportable makeTransportable() {
+  @Override
+public synchronized Transportable makeTransportable() {
     return new CASTransportable(mCasPool, new OutOfTypeSystemData(), null, false);
   }
 
diff --git a/uimaj-core/src/main/java/org/apache/uima/cas/FSIndex.java b/uimaj-core/src/main/java/org/apache/uima/cas/FSIndex.java
index cae79a2..4838f1d 100644
--- a/uimaj-core/src/main/java/org/apache/uima/cas/FSIndex.java
+++ b/uimaj-core/src/main/java/org/apache/uima/cas/FSIndex.java
@@ -62,8 +62,7 @@ package org.apache.uima.cas;
  * Indexes have a top-most type, either the top of the type hierarchy, or some subtype of that.
  * The top-most type in an index is represented by the generic T.
  * 
- * @param T the topmost type in this Index
- * 
+ * @param <T> the topmost type in this Index
  */
 public interface FSIndex<T extends FeatureStructure> extends Iterable<T> {
 
diff --git a/uimaj-core/src/main/java/org/apache/uima/jcas/JCas.java b/uimaj-core/src/main/java/org/apache/uima/jcas/JCas.java
index eef350c..050e524 100644
--- a/uimaj-core/src/main/java/org/apache/uima/jcas/JCas.java
+++ b/uimaj-core/src/main/java/org/apache/uima/jcas/JCas.java
@@ -46,6 +46,7 @@ import org.apache.uima.cas.impl.LowLevelCAS;
 import org.apache.uima.cas.impl.LowLevelIndexRepository;
 import org.apache.uima.cas.text.AnnotationIndex;
 import org.apache.uima.jcas.cas.FSArray;
+import org.apache.uima.jcas.cas.FloatArray;
 import org.apache.uima.jcas.cas.IntegerArray;
 import org.apache.uima.jcas.cas.Sofa;
 import org.apache.uima.jcas.cas.StringArray;
@@ -250,6 +251,7 @@ public interface JCas extends AbstractCas {
    * avoid creating multiple copies of it. All uses can use the same value because it is not
    * updatable (it has no subfields). This is initialized lazily on first reference, and reset when
    * the CAS is reset.
+   * 
    * @return 0-length instance of an IntegerArray
    */
   public abstract IntegerArray getIntegerArray0L();
@@ -260,6 +262,7 @@ public interface JCas extends AbstractCas {
    * updatable (it has no subfields). This is initialized lazily on first reference, and reset when
    * the CAS is reset.
    * @return 0-length instance of a FloatArray
+   */
   public abstract FloatArray getFloatArray0L();
 
   /**
@@ -267,6 +270,7 @@ public interface JCas extends AbstractCas {
    * avoid creating multiple copies of it. All uses can use the same value because it is not
    * updatable (it has no subfields). This is initialized lazily on first reference, and reset when
    * the CAS is reset.
+   * 
    * @return 0-length instance of a FSArray
    */
   public abstract FSArray getFSArray0L();
diff --git a/uimaj-cpe/src/main/java/org/apache/uima/collection/impl/cpm/engine/ProcessingUnit.java b/uimaj-cpe/src/main/java/org/apache/uima/collection/impl/cpm/engine/ProcessingUnit.java
index 465385b..2a86345 100644
--- a/uimaj-cpe/src/main/java/org/apache/uima/collection/impl/cpm/engine/ProcessingUnit.java
+++ b/uimaj-cpe/src/main/java/org/apache/uima/collection/impl/cpm/engine/ProcessingUnit.java
@@ -477,7 +477,8 @@ public class ProcessingUnit extends Thread {
    * configured Work Queue. Once the CAS arrives, it is removed from the queue and sent through the
    * analysis pipeline.
    */
-  public void run() {
+  @Override
+public void run() {
     if (!cpm.isRunning()) {
 
       UIMAFramework.getLogger(this.getClass()).logrb(Level.WARNING, this.getClass().getName(),
@@ -1782,6 +1783,7 @@ public class ProcessingUnit extends Thread {
   }
 
   /**
+   * @param aStatus -
    * @return true if the CASProcessor status is available for processing
    */
   protected boolean isProcessorReady(int aStatus) {
@@ -1795,8 +1797,7 @@ public class ProcessingUnit extends Thread {
   /**
    * Returns the size of the CAS object. Currently only CASData is supported.
    * 
-   * @param aCas -
-   *          Cas to get the size for
+   * @param aCas CAS to get the size for
    * 
    * @return the size of the CAS object. Currently only CASData is supported.
    */
diff --git a/uimaj-cpe/src/main/java/org/apache/uima/collection/impl/cpm/utils/ChunkMetadata.java b/uimaj-cpe/src/main/java/org/apache/uima/collection/impl/cpm/utils/ChunkMetadata.java
index d7a4c51..30dc714 100644
--- a/uimaj-cpe/src/main/java/org/apache/uima/collection/impl/cpm/utils/ChunkMetadata.java
+++ b/uimaj-cpe/src/main/java/org/apache/uima/collection/impl/cpm/utils/ChunkMetadata.java
@@ -84,9 +84,6 @@ public class ChunkMetadata {
     return timedOut;
   }
 
-  /**
-   * @param b
-   */
   public void setTimedOut(boolean b) {
     timedOut = b;
   }
diff --git a/uimaj-cpe/src/main/java/org/apache/uima/collection/impl/cpm/vinci/DATACasUtils.java b/uimaj-cpe/src/main/java/org/apache/uima/collection/impl/cpm/vinci/DATACasUtils.java
index b2e4ad6..3dc6f88 100644
--- a/uimaj-cpe/src/main/java/org/apache/uima/collection/impl/cpm/vinci/DATACasUtils.java
+++ b/uimaj-cpe/src/main/java/org/apache/uima/collection/impl/cpm/vinci/DATACasUtils.java
@@ -57,13 +57,6 @@ public class DATACasUtils {
     return new String(baos.toByteArray());
   }
 
-  /**
-   * 
-   * @param dataCas -
-   * @param featureType -
-   * @param featureName -
-   * @param featureValue -
-   */
   public static void addFeatureStructure(CasData dataCas, String featureType, String featureName,
           String featureValue) {
     FeatureStructure vfs = new FeatureStructureImpl();
@@ -86,13 +79,6 @@ public class DATACasUtils {
     return false;
   }
 
-  /**
-   * 
-   * @param dataCas -
-   * @param featureType -
-   * @param featureName -
-   * @param featureValue -
-   */
   public static void addFeature(CasData dataCas, String featureType, String featureName,
           String featureValue) {
     Iterator it = dataCas.getFeatureStructures();
@@ -199,12 +185,6 @@ public class DATACasUtils {
     return false;
   }
 
-  /**
-   * 
-   * @param aKey -
-   * @param typeList -
-   * @return true if tbd 
-   */
   public static boolean isValidType(String aKey, String[] typeList) {
 
     if (aKey.indexOf(org.apache.uima.collection.impl.cpm.Constants.SHORT_DASH_TERM) > -1) {
@@ -226,12 +206,6 @@ public class DATACasUtils {
     return false;
   }
 
-  /**
-   * 
-   * @param aCAS -
-   * @param featureName -
-   * @return true if
-   */
   public static boolean hasFeature(CasData aCAS, String featureName) {
     Iterator it = aCAS.getFeatureStructures();
     while (it.hasNext()) {
@@ -247,12 +221,6 @@ public class DATACasUtils {
     return false;
   }
 
-  /**
-   * 
-   * @param aCAS -
-   * @param aName -
-   * @return true if tbd
-   */
   public static boolean hasFeatureStructure(CasData aCAS, String aName) {
     Iterator it = aCAS.getFeatureStructures();
     while (it.hasNext()) {
@@ -269,10 +237,6 @@ public class DATACasUtils {
     return false;
   }
 
-  /**
-   * 
-   * @param aCAS -
-   */
   public static void dumpFeatures(CasData aCAS) {
     Iterator it = aCAS.getFeatureStructures();
     while (it.hasNext()) {
@@ -308,12 +272,6 @@ public class DATACasUtils {
 
   }
 
-  /**
-   * 
-   * @param aCAS -
-   * @param featureName -
-   * @return true if tbd
-   */
   public static String getFeatureValueByType(CasData aCAS, String featureName) {
     Iterator it = aCAS.getFeatureStructures();
     String featureValue = null;
@@ -364,13 +322,6 @@ public class DATACasUtils {
     return featureValue;
   }
 
-  /**
-   * 
-   * @param aCAS
-   * @param featureStructureName
-   * @param featureName
-   * @return tbd
-   */
   public static String[] getFeatureStructureValues(CasData aCAS, String featureStructureName,
           String featureName) {
     Iterator it = aCAS.getFeatureStructures();
@@ -400,13 +351,6 @@ public class DATACasUtils {
     return features;
   }
 
-  /**
-   * 
-   * @param aCAS
-   * @param aFeatureStructure
-   * @param featureName
-   * @return tbd
-   */
   public static String getFeatureValueByType(CasData aCAS, String aFeatureStructure,
           String featureName) {
     Iterator it = aCAS.getFeatureStructures();
@@ -429,11 +373,6 @@ public class DATACasUtils {
     return featureValue;
   }
 
-  /**
-   * 
-   * @param aDataCas
-   * @param aFeatureMap
-   */
   public static void remapFeatureTypes(CasData aDataCas, FeatureMap aFeatureMap) {
     ConfigurableFeature cf = null;
 
@@ -459,12 +398,6 @@ public class DATACasUtils {
     }
   }
 
-  /**
-   * 
-   * @param aCasData
-   * @param aFeatureStructureName
-   * @return tbd
-   */
   public static NameValuePair[] getCasDataFeatures(CasData aCasData, String aFeatureStructureName) {
     NameValuePair[] valuePairSet = null;
     Iterator it = aCasData.getFeatureStructures();
diff --git a/uimaj-cpe/src/main/java/org/apache/uima/collection/impl/cpm/vinci/Vinci.java b/uimaj-cpe/src/main/java/org/apache/uima/collection/impl/cpm/vinci/Vinci.java
index ab60967..9580852 100644
--- a/uimaj-cpe/src/main/java/org/apache/uima/collection/impl/cpm/vinci/Vinci.java
+++ b/uimaj-cpe/src/main/java/org/apache/uima/collection/impl/cpm/vinci/Vinci.java
@@ -39,6 +39,7 @@ public class Vinci {
   public static final String VNS_PORT = "VNS_PORT";
 
   public static class AFFactory implements TransportableFactory {
+    @Override
     public Transportable makeTransportable() {
       return new AFrame();
     }
@@ -88,9 +89,10 @@ public class Vinci {
    * 
    * @param conn the connection
    * @param requestFrame {@link org.apache.vinci.transport.VinciFrame}
-   *          containing result of thsi service operation
+   *          containing result of this service operation
    * 
    * @return {@link org.apache.vinci.transport.VinciFrame} VinciData frame.
+   * @throws Exception -
    */
   public static AFrame replyWithAnalysis(VinciClient conn, VinciFrame requestFrame)
           throws Exception {
@@ -114,8 +116,9 @@ public class Vinci {
 
   public static String extractKEYSAsString(AFrame frame) {
     String keys = "";
-    if (frame == null)
-      return keys;
+    if (frame == null) {
+        return keys;
+    }
 
     String frameAsString = frame.toXML();
     if (frameAsString.indexOf("KEYS") > -1 && frameAsString.indexOf("</KEYS>") > -1) {
@@ -156,8 +159,8 @@ public class Vinci {
   }
 
   /**
-   * Returns a content from a given VinciFrame.
-   * 
+   * @param aFrame -
+   * @return a content from a given VinciFrame.
    */
   public static String stripVinciFrame(VinciFrame aFrame) {
     String contentFrame = aFrame.toXML();
diff --git a/uimaj-cpe/src/main/java/org/apache/uima/collection/impl/metadata/cpe/CasProcessorCpeObject.java b/uimaj-cpe/src/main/java/org/apache/uima/collection/impl/metadata/cpe/CasProcessorCpeObject.java
index c7ac970..4fc4585 100644
--- a/uimaj-cpe/src/main/java/org/apache/uima/collection/impl/metadata/cpe/CasProcessorCpeObject.java
+++ b/uimaj-cpe/src/main/java/org/apache/uima/collection/impl/metadata/cpe/CasProcessorCpeObject.java
@@ -121,6 +121,7 @@ public class CasProcessorCpeObject extends MetaDataObject_impl implements CpeCas
    * 
    * @return String - deployment type
    */
+  @Override
   public String getDeployment() {
     return deployment;
   }
@@ -132,6 +133,7 @@ public class CasProcessorCpeObject extends MetaDataObject_impl implements CpeCas
    *          path to the descriptor
    * @throws CpeDescriptorException tbd
    */
+  @Override
   public void setDescriptor(String aDescriptorPath) throws CpeDescriptorException {
     if (descriptor == null) {
       CpeComponentDescriptor comp_desc = CpeDescriptorFactory
@@ -149,6 +151,7 @@ public class CasProcessorCpeObject extends MetaDataObject_impl implements CpeCas
    * @deprecated Doesn't support the new import syntax.  Use getCpeComponentDescriptor().findAbsoluteUrl() instead.
    */
 
+  @Override
   @Deprecated
 public String getDescriptor() {
     if (descriptor != null && descriptor.getInclude() != null) {
@@ -163,6 +166,7 @@ public String getDescriptor() {
    * 
    * @return {@link CpeComponentDescriptor} instance
    */
+  @Override
   public CpeComponentDescriptor getCpeComponentDescriptor() {
     return descriptor;
   }
@@ -174,6 +178,7 @@ public String getDescriptor() {
    * @param aFilterExpression -
    *          String containing a filter
    */
+  @Override
   public void setCasProcessorFilter(String aFilterExpression) {
     if (filter == null) {
       filter = CpeDescriptorFactory.produceCasProcessorFilter("");
@@ -186,6 +191,7 @@ public String getDescriptor() {
    * 
    * @return String - a filter string
    */
+  @Override
   public String getCasProcessorFilter() {
 
     if (getFilter() == null || getFilter().getFilterString() == null) {
@@ -250,6 +256,7 @@ public String getDescriptor() {
    * @param aBatchSize -
    *          batch size of this CasProcessor
    */
+  @Override
   public void setBatchSize(int aBatchSize) {
     checkpoint.setBatchSize(aBatchSize);
   }
@@ -259,6 +266,7 @@ public String getDescriptor() {
    * 
    * @return - batch size as int, 0 if not defined.
    */
+  @Override
   public int getBatchSize() {
     return checkpoint.getBatchSize();
   }
@@ -297,6 +305,7 @@ public String getDescriptor() {
    * 
    * @throws CpeDescriptorException tbd
    */
+  @Override
   public void addDeployParam(String aParamName, String aParamValue) throws CpeDescriptorException {
     boolean found = false;
 
@@ -314,11 +323,6 @@ public String getDescriptor() {
     }
   }
 
-  /**
-   * 
-   * @param aParams
-   * @throws CpeDescriptorException tbd
-   */
   protected void setDeploymentParams(CasProcessorDeploymentParams aParams)
           throws CpeDescriptorException {
     deploymentParameters = aParams;
@@ -330,6 +334,7 @@ public String getDescriptor() {
    * @return the deployment parameters
    * @see org.apache.uima.collection.metadata.CasProcessorDeploymentParams instance
    */
+  @Override
   public CasProcessorDeploymentParams getDeploymentParams() {
     return deploymentParameters;
   }
@@ -342,6 +347,7 @@ public String getDescriptor() {
    * 
    * @throws CpeDescriptorException tbd
    */
+  @Override
   public void setName(String aName) throws CpeDescriptorException {
 
     if (aName == null || aName.trim().length() == 0) {
@@ -358,15 +364,12 @@ public String getDescriptor() {
    * @return - CasProcessor name as string, null if name undefined
    * 
    */
+  @Override
   public String getName() {
     return name;
   }
 
-  /**
-   * @deprecated
-   * @param aSoFa
-   * @throws CpeDescriptorException tbd
-   */
+  @Override
   @Deprecated
 public void setSOFA(String aSoFa) throws CpeDescriptorException {
     // if (casProcessor != null )
@@ -379,6 +382,7 @@ public void setSOFA(String aSoFa) throws CpeDescriptorException {
    * @deprecated (non-Javadoc)
    * @see org.apache.uima.collection.metadata.CpeCasProcessor#getSOFA()
    */
+  @Override
   @Deprecated
 public String getSOFA() {
 
@@ -394,6 +398,7 @@ public String getSOFA() {
    * 
    * @see org.apache.uima.collection.metadata.CpeCasProcessor#setDescriptorPath(java.lang.String)
    */
+  @Override
   public void setCpeComponentDescriptor(CpeComponentDescriptor aDescriptor)
           throws CpeDescriptorException {
     descriptor = aDescriptor;
@@ -418,6 +423,7 @@ public String getSOFA() {
    * 
    * @see org.apache.uima.collection.metadata.CpeCasProcessor#getErrorHandling()
    */
+  @Override
   public CasProcessorErrorHandling getErrorHandling() {
     return errorHandling;
   }
@@ -431,6 +437,7 @@ public String getSOFA() {
    * @param aErrorCount -
    *          max error tolerance
    */
+  @Override
   public void setMaxErrorCount(int aErrorCount) {
     CasProcessorErrorHandling eh = getErrorHandling();
     if (eh != null && eh.getErrorRateThreshold() != null) {
@@ -446,6 +453,7 @@ public String getSOFA() {
    * 
    * @return max error tolerance
    */
+  @Override
   public int getMaxErrorCount() {
 
     CasProcessorErrorHandling eh = getErrorHandling();
@@ -465,6 +473,7 @@ public String getSOFA() {
    * @param aErrorSampleSize -
    *          max error tolerance
    */
+  @Override
   public void setMaxErrorSampleSize(int aErrorSampleSize) {
 
     CasProcessorErrorHandling eh = getErrorHandling();
@@ -481,6 +490,7 @@ public String getSOFA() {
    * 
    * @return - the sample size
    */
+  @Override
   public int getMaxErrorSampleSize() {
     CasProcessorErrorHandling eh = getErrorHandling();
     if (eh != null && eh.getErrorRateThreshold() != null) {
@@ -514,6 +524,7 @@ public String getSOFA() {
    * @param aAction -
    *          action string
    */
+  @Override
   public void setActionOnMaxError(String aAction) {
     CasProcessorErrorHandling eh = getErrorHandling();
     if (eh != null && eh.getErrorRateThreshold() != null) {
@@ -526,6 +537,7 @@ public String getSOFA() {
    * 
    * @return - action as string ( terminate, continue, disable), null when not defined
    */
+  @Override
   public String getActionOnMaxError() {
     CasProcessorErrorHandling eh = getErrorHandling();
     if (eh != null && eh.getErrorRateThreshold() != null) {
@@ -541,6 +553,7 @@ public String getSOFA() {
    * @param aAction -
    *          action string
    */
+  @Override
   public void setActionOnMaxRestart(String aAction) {
     CasProcessorErrorHandling eh = getErrorHandling();
     if (eh != null && eh.getMaxConsecutiveRestarts() != null) {
@@ -553,6 +566,7 @@ public String getSOFA() {
    * 
    * @return - action as string ( terminate, continue, disable), null when not defined
    */
+  @Override
   public String getActionOnMaxRestart() {
     CasProcessorErrorHandling eh = getErrorHandling();
     if (eh != null && eh.getMaxConsecutiveRestarts() != null) {
@@ -567,6 +581,7 @@ public String getSOFA() {
    * @param aRestartCount -
    *          max number of restarts
    */
+  @Override
   public void setMaxRestartCount(int aRestartCount) {
     CasProcessorErrorHandling eh = getErrorHandling();
     if (eh != null && eh.getMaxConsecutiveRestarts() != null) {
@@ -579,6 +594,7 @@ public String getSOFA() {
    * 
    * @return - restart count as int, 0 if not defined
    */
+  @Override
   public int getMaxRestartCount() {
     CasProcessorErrorHandling eh = getErrorHandling();
     if (eh != null && eh.getMaxConsecutiveRestarts() != null) {
@@ -594,6 +610,7 @@ public String getSOFA() {
    * @param aTimeoutValue -
    *          millis to wait for response
    */
+  @Override
   public void setTimeout(int aTimeoutValue) {
     CasProcessorErrorHandling eh = getErrorHandling();
     if (eh != null && eh.getTimeout() != null) {
@@ -606,6 +623,7 @@ public String getSOFA() {
    * 
    * @return - millis, 0 if not defined
    */
+  @Override
   public int getTimeout() {
     CasProcessorErrorHandling eh = getErrorHandling();
     if (eh != null && eh.getTimeout() != null) {
@@ -620,6 +638,7 @@ public String getSOFA() {
    * @param settings -
    * @throws CpeDescriptorException -
    */
+  @Override
   public void setConfigurationParameterSettings(CasProcessorConfigurationParameterSettings settings)
           throws CpeDescriptorException {
     configurationParameterSettings = settings;
@@ -643,6 +662,7 @@ public String getSOFA() {
   /**
    * Returns configuration parameter settings for this CasProcessor.
    */
+  @Override
   public CasProcessorConfigurationParameterSettings getConfigurationParameterSettings() {
     return configurationParameterSettings;
   }
@@ -671,8 +691,9 @@ public String getSOFA() {
       for (int i = 0; nvp != null && i < nvp.length; i++) {
         local.setParameterValue(nvp[i].getName(), nvp[i].getValue());
       }
-    } else
+    } else {
       local = parameterSettings;
+    }
 
     return local;
 
@@ -716,6 +737,7 @@ public String getSOFA() {
   /**
    * @return container with configuration info for running CasProcessor in separate process
    */
+  @Override
   public CasProcessorRunInSeperateProcess getRunInSeparateProcess() {
     return runInSeparateProcess;
   }
@@ -747,6 +769,7 @@ public String getSOFA() {
    * @see org.apache.uima.resource.metadata.impl.MetaDataObject_impl#buildFromXMLElement(org.w3c.dom.Element,
    *      org.apache.uima.util.XMLParser, org.apache.uima.util.XMLParser.ParsingOptions)
    */
+  @Override
   public void buildFromXMLElement(Element aElement, XMLParser aParser, ParsingOptions aOptions)
           throws InvalidXMLException {
     super.buildFromXMLElement(aElement, aParser, aOptions);
@@ -768,6 +791,7 @@ public String getSOFA() {
    * 
    * @see org.apache.uima.resource.metadata.impl.MetaDataObject_impl#getXMLAttributes()
    */
+  @Override
   protected AttributesImpl getXMLAttributes() {
     // TODO Auto-generated method stub
     AttributesImpl attrs = super.getXMLAttributes();
@@ -777,6 +801,7 @@ public String getSOFA() {
     return attrs;
   }
 
+  @Override
   protected XmlizationInfo getXmlizationInfo() {
     return XMLIZATION_INFO;
   }
@@ -793,6 +818,7 @@ public String getSOFA() {
   /**
    * @return configuration for a checkpoint
    */
+  @Override
   public CpeCheckpoint getCheckpoint() {
     return checkpoint;
   }
@@ -800,6 +826,7 @@ public String getSOFA() {
   /**
    * @return sofa name mappings
    */
+  @Override
   public CpeSofaMappings getSofaNameMappings() {
     return sofaNameMappings;
   }
@@ -807,14 +834,17 @@ public String getSOFA() {
   /**
    * @param mappings sofa name mappings
    */
+  @Override
   public void setSofaNameMappings(CpeSofaMappings mappings) {
     sofaNameMappings = mappings;
   }
 
+  @Override
   public void setIsParallelizable(boolean isP) {
     isParallelizable = isP;
   }
 
+  @Override
   public boolean getIsParallelizable() {
     return isParallelizable;
   }
diff --git a/uimaj-cpe/src/main/java/org/apache/uima/collection/impl/metadata/cpe/CpeCasProcessorsImpl.java b/uimaj-cpe/src/main/java/org/apache/uima/collection/impl/metadata/cpe/CpeCasProcessorsImpl.java
index 399324f..c4244a2 100644
--- a/uimaj-cpe/src/main/java/org/apache/uima/collection/impl/metadata/cpe/CpeCasProcessorsImpl.java
+++ b/uimaj-cpe/src/main/java/org/apache/uima/collection/impl/metadata/cpe/CpeCasProcessorsImpl.java
@@ -66,6 +66,7 @@ public class CpeCasProcessorsImpl extends MetaDataObject_impl implements CpeCasP
    * 
    * @see org.apache.uima.collection.metadata.CpeCasProcessors#setOutputQueueSize(int)
    */
+  @Override
   public void setOutputQueueSize(int aOutputQueueSize) throws CpeDescriptorException {
     outputQueueSize = aOutputQueueSize;
   }
@@ -75,6 +76,7 @@ public class CpeCasProcessorsImpl extends MetaDataObject_impl implements CpeCasP
    * 
    * @see org.apache.uima.collection.metadata.CpeCasProcessors#getOutputQueueSize()
    */
+  @Override
   public int getOutputQueueSize() {
     return outputQueueSize;
   }
@@ -84,6 +86,7 @@ public class CpeCasProcessorsImpl extends MetaDataObject_impl implements CpeCasP
    * 
    * @see org.apache.uima.collection.metadata.CpeCasProcessors#setInputQueueSize(int)
    */
+  @Override
   public void setInputQueueSize(int aInputQueueSize) throws CpeDescriptorException {
     inputQueueSize = aInputQueueSize;
   }
@@ -93,6 +96,7 @@ public class CpeCasProcessorsImpl extends MetaDataObject_impl implements CpeCasP
    * 
    * @see org.apache.uima.collection.metadata.CpeCasProcessors#getInputQueueSize()
    */
+  @Override
   public int getInputQueueSize() {
     return inputQueueSize;
   }
@@ -102,6 +106,7 @@ public class CpeCasProcessorsImpl extends MetaDataObject_impl implements CpeCasP
    * 
    * @see org.apache.uima.collection.metadata.CpeCasProcessors#setConcurrentPUCount(int)
    */
+  @Override
   public void setConcurrentPUCount(int aConcurrentPUCount) throws CpeDescriptorException {
     processingUnitThreadCount = aConcurrentPUCount;
   }
@@ -111,6 +116,7 @@ public class CpeCasProcessorsImpl extends MetaDataObject_impl implements CpeCasP
    * 
    * @see org.apache.uima.collection.metadata.CpeCasProcessors#getConcurrentPUCount()
    */
+  @Override
   public int getConcurrentPUCount() {
     return processingUnitThreadCount;
   }
@@ -121,6 +127,7 @@ public class CpeCasProcessorsImpl extends MetaDataObject_impl implements CpeCasP
    * @see org.apache.uima.collection.metadata.CpeCasProcessors#addCpeCasProcessor(org.apache.uima.collection.metadata.CpeCasProcessor,
    *      int)
    */
+  @Override
   public void addCpeCasProcessor(CpeCasProcessor aCasProcessor, int aInsertPosition)
           throws CpeDescriptorException {
     casProcessors.add(aInsertPosition, aCasProcessor);
@@ -132,6 +139,7 @@ public class CpeCasProcessorsImpl extends MetaDataObject_impl implements CpeCasP
    * @see org.apache.uima.collection.metadata.CpeCasProcessors#addCpeCasProcessor(org.apache.uima.collection.metadata.CpeCasProcessor,
    *      int)
    */
+  @Override
   public void addCpeCasProcessor(CpeCasProcessor aCasProcessor) throws CpeDescriptorException {
     casProcessors.add(aCasProcessor);
   }
@@ -141,6 +149,7 @@ public class CpeCasProcessorsImpl extends MetaDataObject_impl implements CpeCasP
    * 
    * @see org.apache.uima.collection.metadata.CpeCasProcessors#getCpeCasProcessor(int)
    */
+  @Override
   public CpeCasProcessor getCpeCasProcessor(int aPosition) throws CpeDescriptorException {
     if (aPosition <= casProcessors.size()) {
       return (CpeCasProcessor) casProcessors.get(aPosition);
@@ -156,6 +165,7 @@ public class CpeCasProcessorsImpl extends MetaDataObject_impl implements CpeCasP
    * 
    * @see org.apache.uima.collection.metadata.CpeCasProcessors#getAllCpeCasProcessors()
    */
+  @Override
   public CpeCasProcessor[] getAllCpeCasProcessors() throws CpeDescriptorException {
     CpeCasProcessor[] processors = new CpeCasProcessor[casProcessors.size()];
     casProcessors.toArray(processors);
@@ -180,6 +190,7 @@ public class CpeCasProcessorsImpl extends MetaDataObject_impl implements CpeCasP
    * 
    * @see org.apache.uima.collection.metadata.CpeCasProcessors#removeCpeCasProcessor(int)
    */
+  @Override
   public void removeCpeCasProcessor(int aPosition) throws CpeDescriptorException {
     if (aPosition <= casProcessors.size()) {
       casProcessors.remove(aPosition);
@@ -193,6 +204,10 @@ public class CpeCasProcessorsImpl extends MetaDataObject_impl implements CpeCasP
 
   /**
    * New API 01/06/2006
+   * @param aPosition -
+   * @param flag -
+   * @return -
+   * @throws CpeDescriptorException -
    */
   public CpeCasProcessor[] removeCpeCasProcessor(int aPosition, boolean flag)
           throws CpeDescriptorException {
@@ -212,6 +227,7 @@ public class CpeCasProcessorsImpl extends MetaDataObject_impl implements CpeCasP
    * 
    * @see org.apache.uima.collection.metadata.CpeCasProcessors#removeAllCpeCasProcessors()
    */
+  @Override
   public void removeAllCpeCasProcessors() throws CpeDescriptorException {
     casProcessors.clear();
   }
@@ -221,6 +237,7 @@ public class CpeCasProcessorsImpl extends MetaDataObject_impl implements CpeCasP
    * 
    * @see org.apache.uima.collection.metadata.CpeCasProcessors#setOutputQueueSize(int)
    */
+  @Override
   public void setPoolSize(int aPoolSize) throws CpeDescriptorException {
     casPoolSize = aPoolSize;
   }
@@ -248,6 +265,7 @@ public class CpeCasProcessorsImpl extends MetaDataObject_impl implements CpeCasP
    * 
    * @see org.apache.uima.collection.metadata.CpeCasProcessors#setOutputQueueSize(int)
    */
+  @Override
   public boolean getDropCasOnException() {
     return dropCasOnException;
   }
@@ -258,6 +276,7 @@ public class CpeCasProcessorsImpl extends MetaDataObject_impl implements CpeCasP
    * @see org.apache.uima.resource.metadata.impl.MetaDataObject_impl#buildFromXMLElement(org.w3c.dom.Element,
    *      org.apache.uima.util.XMLParser, org.apache.uima.util.XMLParser.ParsingOptions)
    */
+  @Override
   public void buildFromXMLElement(Element aElement, XMLParser aParser, ParsingOptions aOptions)
           throws InvalidXMLException {
     try {
@@ -359,6 +378,7 @@ public class CpeCasProcessorsImpl extends MetaDataObject_impl implements CpeCasP
     }
   }
 
+  @Override
   public void toXML(ContentHandler aContentHandler, boolean aWriteDefaultNamespaceAttribute)
           throws SAXException {
     XmlizationInfo inf = getXmlizationInfo();
@@ -391,6 +411,7 @@ public class CpeCasProcessorsImpl extends MetaDataObject_impl implements CpeCasP
    * 
    * @see org.apache.uima.resource.metadata.impl.MetaDataObject_impl#getXMLAttributes()
    */
+  @Override
   protected AttributesImpl getXMLAttributes() {
     AttributesImpl attrs = super.getXMLAttributes();
     if (isDropCasOnException() == true) {
@@ -412,6 +433,7 @@ public class CpeCasProcessorsImpl extends MetaDataObject_impl implements CpeCasP
     return attrs;
   }
 
+  @Override
   protected XmlizationInfo getXmlizationInfo() {
     return XMLIZATION_INFO;
   }
@@ -419,6 +441,7 @@ public class CpeCasProcessorsImpl extends MetaDataObject_impl implements CpeCasP
   static final private XmlizationInfo XMLIZATION_INFO = new XmlizationInfo("casProcessors",
           new PropertyXmlInfo[] { new PropertyXmlInfo("allCpeCasProcessors", null), });
 
+  @Override
   public int getCasPoolSize() {
     // Indirection needed to handle the exception. xsdbeans based implementation of this API
     // was throwing it, if there was a problem. With the current approach this exception never
diff --git a/uimaj-cpe/src/main/java/org/apache/uima/collection/impl/metadata/cpe/CpeCheckpointImpl.java b/uimaj-cpe/src/main/java/org/apache/uima/collection/impl/metadata/cpe/CpeCheckpointImpl.java
index 131038d..535f3d8 100644
--- a/uimaj-cpe/src/main/java/org/apache/uima/collection/impl/metadata/cpe/CpeCheckpointImpl.java
+++ b/uimaj-cpe/src/main/java/org/apache/uima/collection/impl/metadata/cpe/CpeCheckpointImpl.java
@@ -47,6 +47,7 @@ public class CpeCheckpointImpl extends MetaDataObject_impl implements CpeCheckpo
    * 
    * @see org.apache.uima.collection.metadata.CpeCheckpoint#setFilePath(java.lang.String)
    */
+  @Override
   public void setFilePath(String aCheckpointFilePath) throws CpeDescriptorException {
     file = aCheckpointFilePath;
   }
@@ -56,6 +57,7 @@ public class CpeCheckpointImpl extends MetaDataObject_impl implements CpeCheckpo
    * 
    * @see org.apache.uima.collection.metadata.CpeCheckpoint#getFilePath()
    */
+  @Override
   public String getFilePath() {
     return file;
   }
@@ -65,6 +67,7 @@ public class CpeCheckpointImpl extends MetaDataObject_impl implements CpeCheckpo
    * 
    * @see org.apache.uima.collection.metadata.CpeCheckpoint#setFrequency(int, boolean)
    */
+  @Override
   public void setFrequency(int aFrequency, boolean aTimeBased) {
     time = String.valueOf(aFrequency) + "ms";
   }
@@ -76,10 +79,12 @@ public class CpeCheckpointImpl extends MetaDataObject_impl implements CpeCheckpo
       int len = ((String) anObject).length();
       int i = 0;
       for (; i < len; i++) {
-        if (Character.isDigit(((String) anObject).charAt(i)))
+        if (Character.isDigit(((String) anObject).charAt(i))) {
           continue;
-        else
+        }
+        else {
           break; // non-digit char
+        }
       }
       if (i > 0) {
         convertedTime = Integer.parseInt(((String) anObject).substring(0, i));
@@ -94,6 +99,7 @@ public class CpeCheckpointImpl extends MetaDataObject_impl implements CpeCheckpo
    * 
    * @see org.apache.uima.collection.metadata.CpeCheckpoint#getFrequency()
    */
+  @Override
   public int getFrequency() {
     return convert2Number(time);
   }
@@ -103,6 +109,7 @@ public class CpeCheckpointImpl extends MetaDataObject_impl implements CpeCheckpo
    * 
    * @see org.apache.uima.collection.metadata.CpeCheckpoint#isTimeBased()
    */
+  @Override
   public boolean isTimeBased() {
     return true;
   }
@@ -112,6 +119,7 @@ public class CpeCheckpointImpl extends MetaDataObject_impl implements CpeCheckpo
    * 
    * @see org.apache.uima.collection.metadata.CpeCheckpoint#setBatchSize(int)
    */
+  @Override
   public void setBatchSize(int aBatchSize) {
     batch = aBatchSize;
   }
@@ -121,6 +129,7 @@ public class CpeCheckpointImpl extends MetaDataObject_impl implements CpeCheckpo
    * 
    * @see org.apache.uima.collection.metadata.CpeCheckpoint#getBatchSize()
    */
+  @Override
   public int getBatchSize() {
     return batch;
   }
@@ -131,6 +140,7 @@ public class CpeCheckpointImpl extends MetaDataObject_impl implements CpeCheckpo
    * @see org.apache.uima.resource.metadata.impl.MetaDataObject_impl#buildFromXMLElement(org.w3c.dom.Element,
    *      org.apache.uima.util.XMLParser, org.apache.uima.util.XMLParser.ParsingOptions)
    */
+  @Override
   public void buildFromXMLElement(Element aElement, XMLParser aParser, ParsingOptions aOptions)
           throws InvalidXMLException {
     setFile(aElement.getAttribute("file"));
@@ -147,6 +157,7 @@ public class CpeCheckpointImpl extends MetaDataObject_impl implements CpeCheckpo
    * 
    * @see org.apache.uima.resource.metadata.impl.MetaDataObject_impl#getXMLAttributes()
    */
+  @Override
   protected AttributesImpl getXMLAttributes() {
     AttributesImpl attrs = super.getXMLAttributes();
     attrs.addAttribute("", "batch", "batch", "CDATA", String.valueOf(getBatch()));
@@ -159,6 +170,7 @@ public class CpeCheckpointImpl extends MetaDataObject_impl implements CpeCheckpo
     return attrs;
   }
 
+  @Override
   protected XmlizationInfo getXmlizationInfo() {
     return XMLIZATION_INFO;
   }
@@ -166,7 +178,8 @@ public class CpeCheckpointImpl extends MetaDataObject_impl implements CpeCheckpo
   static final private XmlizationInfo XMLIZATION_INFO = new XmlizationInfo("checkpoint",
           new PropertyXmlInfo[0]);
 
-  /** METHODS CALLED BY THE PARSER * */
+  // METHODS CALLED BY THE PARSER
+  
   /**
    * @return the batch size
    */
@@ -188,23 +201,14 @@ public class CpeCheckpointImpl extends MetaDataObject_impl implements CpeCheckpo
     return time;
   }
 
-  /**
-   * @param i
-   */
   public void setBatch(int i) {
     batch = i;
   }
 
-  /**
-   * @param string
-   */
   public void setFile(String string) {
     file = string;
   }
 
-  /**
-   * @param i
-   */
   public void setTime(String i) {
     time = i;
   }
diff --git a/uimaj-cpe/src/main/java/org/apache/uima/collection/impl/metadata/cpe/CpeCollectionReaderCasInitializerImpl.java b/uimaj-cpe/src/main/java/org/apache/uima/collection/impl/metadata/cpe/CpeCollectionReaderCasInitializerImpl.java
index 93bc789..5acde26 100644
--- a/uimaj-cpe/src/main/java/org/apache/uima/collection/impl/metadata/cpe/CpeCollectionReaderCasInitializerImpl.java
+++ b/uimaj-cpe/src/main/java/org/apache/uima/collection/impl/metadata/cpe/CpeCollectionReaderCasInitializerImpl.java
@@ -56,6 +56,7 @@ public class CpeCollectionReaderCasInitializerImpl extends MetaDataObject_impl i
    * 
    * @see org.apache.uima.collection.metadata.CpeCollectionReaderCasInitializer#setDescriptorPath(java.lang.String)
    */
+  @Override
   public void setDescriptor(CpeComponentDescriptor aDescriptor) {
     descriptor = aDescriptor;
   }
@@ -65,6 +66,7 @@ public class CpeCollectionReaderCasInitializerImpl extends MetaDataObject_impl i
    * 
    * @see org.apache.uima.collection.metadata.CpeCollectionReaderCasInitializer#getDescriptorPath()
    */
+  @Override
   public CpeComponentDescriptor getDescriptor() {
     return descriptor;
   }
@@ -74,6 +76,7 @@ public class CpeCollectionReaderCasInitializerImpl extends MetaDataObject_impl i
    *
    * @return the configuration parameter settings
    */
+  @Override
   public CasProcessorConfigurationParameterSettings getConfigurationParameterSettings() {
     return cfps;
   }
@@ -119,16 +122,14 @@ public class CpeCollectionReaderCasInitializerImpl extends MetaDataObject_impl i
       for (int i = 0; nvp != null && i < nvp.length; i++) {
         local.setParameterValue(nvp[i].getName(), nvp[i].getValue());
       }
-    } else
+    } else {
       local = configurationParameterSettings;
+    }
 
     return local;
 
   }
 
-  /**
-   * @param settings
-   */
   public void setParameterSettings(ConfigurationParameterSettings settings) {
     configurationParameterSettings = settings;
     if (configurationParameterSettings != null) {
@@ -137,6 +138,7 @@ public class CpeCollectionReaderCasInitializerImpl extends MetaDataObject_impl i
 
   }
 
+  @Override
   protected XmlizationInfo getXmlizationInfo() {
     return XMLIZATION_INFO;
   }
@@ -145,21 +147,18 @@ public class CpeCollectionReaderCasInitializerImpl extends MetaDataObject_impl i
           new PropertyXmlInfo[] { new PropertyXmlInfo("descriptor", null),
               new PropertyXmlInfo("parameterSettings", null),
               new PropertyXmlInfo("sofaNameMappings", null),
-
           });
 
   /**
    * @return the sofa mappings
    */
+  @Override
   public CpeSofaMappings getSofaNameMappings() {
     return sofaNameMappings;
   }
 
-  /**
-   * @param mappings
-   */
+  @Override
   public void setSofaNameMappings(CpeSofaMappings mappings) {
     sofaNameMappings = mappings;
   }
-
 }
diff --git a/uimaj-ep-cas-editor-ide/src/main/java/org/apache/uima/caseditor/ide/CasEditorIdePlugin.java b/uimaj-ep-cas-editor-ide/src/main/java/org/apache/uima/caseditor/ide/CasEditorIdePlugin.java
index 2e3f56a..9bf72ab 100644
--- a/uimaj-ep-cas-editor-ide/src/main/java/org/apache/uima/caseditor/ide/CasEditorIdePlugin.java
+++ b/uimaj-ep-cas-editor-ide/src/main/java/org/apache/uima/caseditor/ide/CasEditorIdePlugin.java
@@ -64,7 +64,7 @@ public class CasEditorIdePlugin extends AbstractUIPlugin {
   /**
    * This method is called upon plug-in activation
    *
-   * @param context
+   * @param context -
    * @throws Exception -
    */
   @Override
@@ -111,8 +111,9 @@ public class CasEditorIdePlugin extends AbstractUIPlugin {
 	              try {
 	                DotCorpus dotCorpus = DotCorpusSerializer.parseDotCorpus(dotCorpusIn);
 	                
-	                if (dotCorpus.getTypeSystemFileName() != null)
-	                    typeSystemFile = project.getFile(dotCorpus.getTypeSystemFileName());
+	                if (dotCorpus.getTypeSystemFileName() != null) {
+                        typeSystemFile = project.getFile(dotCorpus.getTypeSystemFileName());
+                    }
 	              }
 	              finally {
 	                try {
@@ -146,7 +147,7 @@ public class CasEditorIdePlugin extends AbstractUIPlugin {
   /**
    * This method is called when the plug-in is stopped.
    *
-   * @param context
+   * @param context -
    * @throws Exception -
    */
   @Override
@@ -169,7 +170,7 @@ public class CasEditorIdePlugin extends AbstractUIPlugin {
   /**
    * Log the throwable.
    *
-   * @param t
+   * @param t -
    */
   public static void log(Throwable t) {
     getDefault().getLog().log(new Status(IStatus.ERROR, ID, IStatus.OK, t.getMessage(), t));
diff --git a/uimaj-ep-configurator/src/main/java/org/apache/uima/taeconfigurator/editors/ui/Utility.java b/uimaj-ep-configurator/src/main/java/org/apache/uima/taeconfigurator/editors/ui/Utility.java
index a36a30b..d83db6a 100644
--- a/uimaj-ep-configurator/src/main/java/org/apache/uima/taeconfigurator/editors/ui/Utility.java
+++ b/uimaj-ep-configurator/src/main/java/org/apache/uima/taeconfigurator/editors/ui/Utility.java
@@ -33,15 +33,6 @@ import org.eclipse.swt.widgets.Widget;
 // MessageDialog extended only to enable resize
 public class Utility extends MessageDialog {
 
-  /**
-   * @param parentShell
-   * @param dialogTitle
-   * @param dialogTitleImage
-   * @param dialogMessage
-   * @param dialogImageType
-   * @param dialogButtonLabels
-   * @param defaultIndex
-   */
   public Utility(Shell parentShell, String dialogTitle, Image dialogTitleImage,
           String dialogMessage, int dialogImageType, String[] dialogButtonLabels, int defaultIndex) {
     super(parentShell, dialogTitle, dialogTitleImage, dialogMessage, dialogImageType,
@@ -58,7 +49,7 @@ public class Utility extends MessageDialog {
    * 
    * @param title
    *          of the warning
-   * @param message
+   * @param message -
    * @param type
    *          one of MessageDialog.NONE for a dialog with no image MessageDialog.ERROR for a dialog
    *          with an error image MessageDialog.INFORMATION for a dialog with an information image
@@ -74,7 +65,7 @@ public class Utility extends MessageDialog {
    * 
    * @param title
    *          of the warning
-   * @param message
+   * @param message -
    * @param type
    *          one of MessageDialog.NONE for a dialog with no image MessageDialog.ERROR for a dialog
    *          with an error image MessageDialog.INFORMATION for a dialog with an information image
@@ -106,7 +97,9 @@ public class Utility extends MessageDialog {
     dialog.setShellStyle(SWT.DIALOG_TRIM | SWT.APPLICATION_MODAL | SWT.RESIZE); 
     int returnCode = dialog.open();
     if (returnCode == -1)
-      returnCode = Window.CANCEL; // Cancel code
+     {
+        returnCode = Window.CANCEL; // Cancel code
+    }
     return returnCode;
   }
 
@@ -117,39 +110,40 @@ public class Utility extends MessageDialog {
   /**
    * remove element(s) (must be present) using == test
    * 
-   * @param source
-   * @param element
-   * @param componentClass
+   * @param source -
+   * @param element -
+   * @param componentClass -
    * @return a copy of the array with == element(s) removed
    */
-
   public static Object[] removeElementFromArray(Object[] source, Object element,
           Class componentClass) {
     Object[] result = (Object[]) Array.newInstance(componentClass, source.length - 1);
     for (int i = 0, j = 0; i < source.length; i++) {
-      if (element != source[i])
+      if (element != source[i]) {
         result[j++] = source[i];
     }
+    }
     return result;
   }
 
   /**
    * remove element(s) (must be present) using equals test
    * 
-   * @param source
-   * @param element
-   * @param componentClass
+   * @param source -
+   * @param element -
+   * @param componentClass -
    * @return a copy of the array with equal element(s) removed
    */
-
   public static Object[] removeEqualElementFromArray(Object[] source, Object element,
           Class componentClass) {
     Object[] result = (Object[]) Array.newInstance(componentClass, source.length - 1);
     for (int i = 0, j = 0; i < source.length; i++) {
-      if (element == null && source[i] == null)
+      if (element == null && source[i] == null) {
         continue;
-      if (null != element && element.equals(source[i]))
+    }
+      if (null != element && element.equals(source[i])) {
         continue;
+    }
       result[j++] = source[i];
     }
     return result;
@@ -157,41 +151,49 @@ public class Utility extends MessageDialog {
 
   public static Object[] removeElementsFromArray(Object[] source, Object element,
           Class componentClass) {
-    if (null == source)
-      return null;
+    if (null == source) {
+        return null;
+    }
     int count = 0;
     for (int i = 0; i < source.length; i++) {
-      if (!element.equals(source[i]))
+      if (!element.equals(source[i])) {
         count++;
     }
-    if (count == source.length)
-      return source;
+    }
+    if (count == source.length) {
+        return source;
+    }
 
     Object[] result = (Object[]) Array.newInstance(componentClass, count);
     for (int i = 0, j = 0; i < source.length; i++) {
-      if (!element.equals(source[i]))
+      if (!element.equals(source[i])) {
         result[j++] = source[i];
     }
+    }
     return result;
   }
 
   public static Object[] removeElementsFromArray(Object[] source, Object element,
           Class componentClass, Comparator comp) {
-    if (null == source)
-      return null;
+    if (null == source) {
+        return null;
+    }
     int count = 0;
     for (int i = 0; i < source.length; i++) {
-      if (0 != comp.compare(element, source[i]))
+      if (0 != comp.compare(element, source[i])) {
         count++;
     }
-    if (count == source.length)
-      return source;
+    }
+    if (count == source.length) {
+        return source;
+    }
 
     Object[] result = (Object[]) Array.newInstance(componentClass, count);
     for (int i = 0, j = 0; i < source.length; i++) {
-      if (0 != comp.compare(element, source[i]))
+      if (0 != comp.compare(element, source[i])) {
         result[j++] = source[i];
     }
+    }
     return result;
   }
 
@@ -204,16 +206,20 @@ public class Utility extends MessageDialog {
   }
 
   public static boolean arrayContains(Object[] array, Object element) {
-    if (null == element)
-      throw new InternalErrorCDE("null not allowed as an argument");
-    if (null == array)
-      return false;
+    if (null == element) {
+        throw new InternalErrorCDE("null not allowed as an argument");
+    }
+    if (null == array) {
+        return false;
+    }
     for (int i = 0; i < array.length; i++) {
-      if (null == array[i])
+      if (null == array[i]) {
         continue;
-      if (array[i].equals(element))
+    }
+      if (array[i].equals(element)) {
         return true;
     }
+    }
     return false;
   }
 
diff --git a/uimaj-ep-launcher/src/main/java/org/apache/uima/ep_launcher/LauncherPlugin.java b/uimaj-ep-launcher/src/main/java/org/apache/uima/ep_launcher/LauncherPlugin.java
index 1659840..e413bf5 100644
--- a/uimaj-ep-launcher/src/main/java/org/apache/uima/ep_launcher/LauncherPlugin.java
+++ b/uimaj-ep-launcher/src/main/java/org/apache/uima/ep_launcher/LauncherPlugin.java
@@ -54,30 +54,33 @@ public class LauncherPlugin extends Plugin {
   public Bundle[] getBundles(String bundleName, String version) {
     
     Bundle[] bundles = Platform.getBundles(bundleName, version);
-    if (bundles != null)
-      return bundles;
+    if (bundles != null) {
+        return bundles;
+    }
 
     // Accessing bundle which is not resolved
     PackageAdmin admin = (PackageAdmin) bundleContext.getService(
             bundleContext.getServiceReference(PackageAdmin.class.getName()));
     bundles = admin.getBundles(bundleName, version);
-    if (bundles != null && bundles.length > 0)
-      return bundles;
+    if (bundles != null && bundles.length > 0) {
+        return bundles;
+    }
     
     return null;
   }
 
   public Bundle getBundle(String bundleName) {
     Bundle[] bundles = getBundles(bundleName, null);
-    if (bundles != null && bundles.length > 0)
-      // return fist bundle, if multiple
-      return bundles[0];
+    if (bundles != null && bundles.length > 0) {
+        // return fist bundle, if multiple
+          return bundles[0];
+    }
     
     return null;
   }
 
   /**
-   * Returns the shared instance.
+   * @return the shared instance.
    */
   public static LauncherPlugin getDefault() {
     return plugin;
diff --git a/uimaj-ep-pear-packager/src/main/java/org/apache/uima/pear/insd/edit/INSDComponentPage.java b/uimaj-ep-pear-packager/src/main/java/org/apache/uima/pear/insd/edit/INSDComponentPage.java
index 6d0f1af..69f9d8e 100644
--- a/uimaj-ep-pear-packager/src/main/java/org/apache/uima/pear/insd/edit/INSDComponentPage.java
+++ b/uimaj-ep-pear-packager/src/main/java/org/apache/uima/pear/insd/edit/INSDComponentPage.java
@@ -73,9 +73,6 @@ public class INSDComponentPage extends WizardPage implements InsdConstants {
 
   Hashtable wizardData;
 
-  /**
-   * Constructor
-   */
   public INSDComponentPage(IContainer currentContainer, InstallationDescriptor insd,
           Hashtable wizardData) {
     super("wizardPage");
@@ -119,10 +116,12 @@ public class INSDComponentPage extends WizardPage implements InsdConstants {
     String temp = "";
 
     temp = insd.getMainComponentId();
-    if (temp == null || temp.trim().length() == 0)
-      compID = currentContainer.getName();
-    else
-      compID = temp;
+    if (temp == null || temp.trim().length() == 0) {
+        compID = currentContainer.getName();
+    }
+    else {
+        compID = temp;
+    }
 
     temp = insd.getMainComponentDesc();
     compDescriptorPath = temp == null ? "" : removeMacros(temp);
@@ -137,7 +136,8 @@ public class INSDComponentPage extends WizardPage implements InsdConstants {
     text.setLayoutData(new GridData(GridData.FILL_HORIZONTAL));
     text.setText(strText);
     text.addModifyListener(new ModifyListener() {
-      public void modifyText(ModifyEvent e) {
+      @Override
+    public void modifyText(ModifyEvent e) {
         dialogChanged();
       }
     });
@@ -148,7 +148,8 @@ public class INSDComponentPage extends WizardPage implements InsdConstants {
   /**
    * See IDialogPage#createControl(Composite)
    */
-  public void createControl(Composite parent) {
+  @Override
+public void createControl(Composite parent) {
 
     try {
 
@@ -210,8 +211,10 @@ public class INSDComponentPage extends WizardPage implements InsdConstants {
 
       setPageComplete(false);
       setErrorMessage(null);
-    } else
-      updateStatus(null);
+    }
+    else {
+        updateStatus(null);
+    }
 
   }
 
@@ -221,8 +224,10 @@ public class INSDComponentPage extends WizardPage implements InsdConstants {
 
       setPageComplete(false);
       setErrorMessage(null);
-    } else
-      updateStatus(null);
+    }
+    else {
+        updateStatus(null);
+    }
   }
 
   void saveWizardData() {
@@ -252,15 +257,18 @@ public class INSDComponentPage extends WizardPage implements InsdConstants {
       if (filename != null && filename.trim().length() > 0) {
         filename = filename.trim();
         IFile iFile = currentContainer.getFile(new Path(filename));
-        if (!iFile.exists())
-          sb.append("\n  \"" + filename + "\" was not found in the current project!");
-        else if (filename.trim().indexOf(".xml") == -1)
-          sb.append("\n  \"" + filename + "\" is not an xml file!");
+        if (!iFile.exists()) {
+            sb.append("\n  \"" + filename + "\" was not found in the current project!");
+        }
+        else if (filename.trim().indexOf(".xml") == -1) {
+            sb.append("\n  \"" + filename + "\" is not an xml file!");
+        }
       }
     }
     String s = sb.toString();
-    if (s.length() > 0)
-      message = s;
+    if (s.length() > 0) {
+        message = s;
+    }
 
     return message;
   }
@@ -282,6 +290,7 @@ public class INSDComponentPage extends WizardPage implements InsdConstants {
    * @param defaultButton
    *          <code>true</code> if the button is to be the default button, and <code>false</code>
    *          otherwise
+   * @param text -
    */
   protected Button addButton(Composite parent, String label, boolean defaultButton, final Text text) {
 
@@ -298,7 +307,8 @@ public class INSDComponentPage extends WizardPage implements InsdConstants {
     button.setFont(parent.getFont());
 
     SelectionListener listener = new SelectionAdapter() {
-      public void widgetSelected(SelectionEvent e) {
+      @Override
+    public void widgetSelected(SelectionEvent e) {
 
         ResourceSelectionDialog dialog = new ResourceSelectionDialog(getShell(), currentContainer,
                 "Selection Dialog");
@@ -337,7 +347,8 @@ public class INSDComponentPage extends WizardPage implements InsdConstants {
     button.setSelection(initialSelection);
 
     SelectionListener listener = new SelectionAdapter() {
-      public void widgetSelected(SelectionEvent e) {
+      @Override
+    public void widgetSelected(SelectionEvent e) {
         dialogChanged();
       }
     };
@@ -348,9 +359,11 @@ public class INSDComponentPage extends WizardPage implements InsdConstants {
   /*
    * see @DialogPage.setVisible(boolean)
    */
-  public void setVisible(boolean visible) {
+  @Override
+public void setVisible(boolean visible) {
     super.setVisible(visible);
-    if (visible)
-      compIDText.setFocus();
+    if (visible) {
+        compIDText.setFocus();
+    }
   }
 }
\ No newline at end of file
diff --git a/uimaj-ep-pear-packager/src/main/java/org/apache/uima/pear/insd/edit/vars/AbstractVarValViewerHandler.java b/uimaj-ep-pear-packager/src/main/java/org/apache/uima/pear/insd/edit/vars/AbstractVarValViewerHandler.java
index e4055c2..e242ff8 100644
--- a/uimaj-ep-pear-packager/src/main/java/org/apache/uima/pear/insd/edit/vars/AbstractVarValViewerHandler.java
+++ b/uimaj-ep-pear-packager/src/main/java/org/apache/uima/pear/insd/edit/vars/AbstractVarValViewerHandler.java
@@ -29,10 +29,6 @@ import org.eclipse.swt.layout.GridData;
 import org.eclipse.swt.layout.GridLayout;
 import org.eclipse.swt.widgets.*;
 
-/**
- * 
- * 
- */
 abstract class AbstractVarValViewerHandler {
 
   public Table table;
@@ -144,8 +140,9 @@ abstract class AbstractVarValViewerHandler {
   public void close() {
     Shell shell = table.getShell();
 
-    if (shell != null && !shell.isDisposed())
-      shell.dispose();
+    if (shell != null && !shell.isDisposed()) {
+        shell.dispose();
+    }
   }
 
   /**
@@ -155,18 +152,23 @@ abstract class AbstractVarValViewerHandler {
    */
   class ExampleContentProvider implements IStructuredContentProvider, IVarValListViewer {
 
+    @Override
     public void inputChanged(Viewer v, Object oldInput, Object newInput) {
-      if (newInput != null)
+      if (newInput != null) {
         ((VarValList) newInput).addChangeListener(this);
-      if (oldInput != null)
+    }
+      if (oldInput != null) {
         ((VarValList) oldInput).removeChangeListener(this);
     }
+    }
 
+    @Override
     public void dispose() {
       tableRowList.removeChangeListener(this);
     }
 
     // Return the tableRows as an array of Objects
+    @Override
     public Object[] getElements(Object parent) {
       return tableRowList.getTableRows().toArray();
     }
@@ -176,6 +178,7 @@ abstract class AbstractVarValViewerHandler {
      * 
      * @see IVarValListViewer#addTableRow(VarVal)
      */
+    @Override
     public void addTableRow(VarVal tableRow) {
       tableViewer.add(tableRow);
     }
@@ -185,6 +188,7 @@ abstract class AbstractVarValViewerHandler {
      * 
      * @see IVarValListViewer#removeTableRow(VarVal)
      */
+    @Override
     public void removeTableRow(VarVal tableRow) {
       tableViewer.remove(tableRow);
     }
@@ -194,6 +198,7 @@ abstract class AbstractVarValViewerHandler {
      * 
      * @see IVarValListViewer#updateTableRow(VarVal)
      */
+    @Override
     public void updateTableRow(VarVal tableRow) {
       tableViewer.update(tableRow, null);
     }
@@ -217,7 +222,8 @@ abstract class AbstractVarValViewerHandler {
     add.addSelectionListener(new SelectionAdapter() {
 
       // Add a tableRow to the VarValList and refresh the view
-      public void widgetSelected(SelectionEvent e) {
+      @Override
+    public void widgetSelected(SelectionEvent e) {
         if (!tableRowList.addTableRow()) {
           MessageDialog.openWarning(new Shell(), "Duplicate Variable",
                   "The variable 'New_Variable' already exists");
@@ -235,7 +241,8 @@ abstract class AbstractVarValViewerHandler {
     delete.addSelectionListener(new SelectionAdapter() {
 
       // Remove the selection and refresh the view
-      public void widgetSelected(SelectionEvent e) {
+      @Override
+    public void widgetSelected(SelectionEvent e) {
         VarVal tableRow = (VarVal) ((IStructuredSelection) tableViewer.getSelection())
                 .getFirstElement();
         if (tableRow != null) {
@@ -263,21 +270,21 @@ abstract class AbstractVarValViewerHandler {
   }
 
   /**
-   * Return the VarValList
+   * @return the VarValList
    */
   public VarValList getTableRowList() {
     return tableRowList;
   }
 
   /**
-   * Return the parent composite
+   * @return the parent composite
    */
   public Control getControl() {
     return table.getParent();
   }
 
   /**
-   * Return the 'close' Button
+   * @return the 'close' Button
    */
   public Button getCloseButton() {
     return closeButton;
diff --git a/uimaj-ep-pear-packager/src/main/java/org/apache/uima/pear/insd/edit/vars/VarValList.java b/uimaj-ep-pear-packager/src/main/java/org/apache/uima/pear/insd/edit/vars/VarValList.java
index 03d6f8f..1393209 100644
--- a/uimaj-ep-pear-packager/src/main/java/org/apache/uima/pear/insd/edit/vars/VarValList.java
+++ b/uimaj-ep-pear-packager/src/main/java/org/apache/uima/pear/insd/edit/vars/VarValList.java
@@ -25,11 +25,7 @@ import java.util.Set;
 import java.util.Vector;
 
 /**
- * 
  * Represents a list of environment variable to be displayed in a table
- * 
- * 
- * 
  */
 public class VarValList {
 
@@ -65,8 +61,10 @@ public class VarValList {
     if (!isDuplicate(tableRow)) {
       tableRows.add(tableRow);
       return true;
-    } else
-      return false;
+    }
+    else {
+        return false;
+    }
   }
 
   /**
@@ -77,11 +75,14 @@ public class VarValList {
     if (!isDuplicate(tableRow)) {
       tableRows.add(tableRows.size(), tableRow);
       Iterator iterator = changeListeners.iterator();
-      while (iterator.hasNext())
+      while (iterator.hasNext()) {
         ((IVarValListViewer) iterator.next()).addTableRow(tableRow);
+    }
       return true;
-    } else
-      return false;
+    }
+    else {
+        return false;
+    }
   }
 
   private boolean isDuplicate(VarVal tableRow) {
@@ -90,48 +91,47 @@ public class VarValList {
     Iterator itr = tableRows.iterator();
     while (itr.hasNext()) {
       VarVal vv = (VarVal) itr.next();
-      if (vv.getVarName().equals(varName))
+      if (vv.getVarName().equals(varName)) {
         duplicate = true;
     }
+    }
     return duplicate;
   }
 
-  /**
-   * @param tableRow
-   */
   public void removeTableRow(VarVal tableRow) {
     tableRows.remove(tableRow);
     Iterator iterator = changeListeners.iterator();
-    while (iterator.hasNext())
-      ((IVarValListViewer) iterator.next()).removeTableRow(tableRow);
+    while (iterator.hasNext()) {
+        ((IVarValListViewer) iterator.next()).removeTableRow(tableRow);
+    }
   }
 
   /**
    * Notify listeners by calling their updateTableRow() method
    * 
-   * @param tableRow
+   * @param tableRow -
    */
   public void tableRowChanged(VarVal tableRow) {
     Iterator iterator = changeListeners.iterator();
-    while (iterator.hasNext())
-      ((IVarValListViewer) iterator.next()).updateTableRow(tableRow);
+    while (iterator.hasNext()) {
+        ((IVarValListViewer) iterator.next()).updateTableRow(tableRow);
+    }
   }
 
   /**
    * Removes a Change Listener
    * 
    * @param viewer
-   *          A Chnage listener
+   *          A change listener
    */
   public void removeChangeListener(IVarValListViewer viewer) {
     changeListeners.remove(viewer);
   }
 
   /**
-   * 
    * Adds a change listener
    * 
-   * @param viewer
+   * @param viewer -
    */
   public void addChangeListener(IVarValListViewer viewer) {
     changeListeners.add(viewer);
diff --git a/uimaj-examples/src/main/java/org/apache/uima/examples/cpe/SimpleRunCPE.java b/uimaj-examples/src/main/java/org/apache/uima/examples/cpe/SimpleRunCPE.java
index 45730b1..333b9e6 100644
--- a/uimaj-examples/src/main/java/org/apache/uima/examples/cpe/SimpleRunCPE.java
+++ b/uimaj-examples/src/main/java/org/apache/uima/examples/cpe/SimpleRunCPE.java
@@ -59,6 +59,7 @@ public class SimpleRunCPE extends Thread {
    * 
    * @param args
    *          command line arguments into the program - see class description
+   * @throws Exception -
    */
   public SimpleRunCPE(String args[]) throws Exception {
     mStartTime = System.currentTimeMillis();
@@ -107,6 +108,7 @@ public class SimpleRunCPE extends Thread {
    * 
    * @param args
    *          Command line arguments - see class description
+   * @throws Exception -
    */
   public static void main(String[] args) throws Exception {
     new SimpleRunCPE(args);
@@ -125,8 +127,9 @@ public class SimpleRunCPE extends Thread {
     /**
      * Called when the initialization is completed.
      * 
-     * @see org.apache.uima.collection.processing.StatusCallbackListener#initializationComplete()
+     * @see StatusCallbackListener#initializationComplete()
      */
+    @Override
     public void initializationComplete() {      
       System.out.println("CPM Initialization Complete");
       mInitCompleteTime = System.currentTimeMillis();
@@ -135,9 +138,10 @@ public class SimpleRunCPE extends Thread {
     /**
      * Called when the batchProcessing is completed.
      * 
-     * @see org.apache.uima.collection.processing.StatusCallbackListener#batchProcessComplete()
+     * @see StatusCallbackListener#batchProcessComplete()
      * 
      */
+    @Override
     public void batchProcessComplete() {
       System.out.print("Completed " + entityCount + " documents");
       if (size > 0) {
@@ -151,8 +155,9 @@ public class SimpleRunCPE extends Thread {
     /**
      * Called when the collection processing is completed.
      * 
-     * @see org.apache.uima.collection.processing.StatusCallbackListener#collectionProcessComplete()
+     * @see StatusCallbackListener#collectionProcessComplete()
      */
+    @Override
     public void collectionProcessComplete() {
       long time = System.currentTimeMillis();
       System.out.print("Completed " + entityCount + " documents");
@@ -177,8 +182,9 @@ public class SimpleRunCPE extends Thread {
     /**
      * Called when the CPM is paused.
      * 
-     * @see org.apache.uima.collection.processing.StatusCallbackListener#paused()
+     * @see StatusCallbackListener#paused()
      */
+    @Override
     public void paused() {
       System.out.println("Paused");
     }
@@ -186,8 +192,9 @@ public class SimpleRunCPE extends Thread {
     /**
      * Called when the CPM is resumed after a pause.
      * 
-     * @see org.apache.uima.collection.processing.StatusCallbackListener#resumed()
+     * @see StatusCallbackListener#resumed()
      */
+    @Override
     public void resumed() {
       System.out.println("Resumed");
     }
@@ -195,8 +202,9 @@ public class SimpleRunCPE extends Thread {
     /**
      * Called when the CPM is stopped abruptly due to errors.
      * 
-     * @see org.apache.uima.collection.processing.StatusCallbackListener#aborted()
+     * @see StatusCallbackListener#aborted()
      */
+    @Override
     public void aborted() {
       System.out.println("Aborted");
       // stop the JVM. Otherwise main thread will still be blocked waiting for
@@ -213,6 +221,7 @@ public class SimpleRunCPE extends Thread {
      * @param aStatus
      *          EntityProcessStatus that holds the status of all the events for aEntity
      */
+    @Override
     public void entityProcessComplete(CAS aCas, EntityProcessStatus aStatus) {
       if (aStatus.isException()) {
         List exceptions = aStatus.getExceptions();
diff --git a/uimaj-examples/src/main/java/org/apache/uima/examples/xmi/Ecore2UimaTypeSystem.java b/uimaj-examples/src/main/java/org/apache/uima/examples/xmi/Ecore2UimaTypeSystem.java
index 4e093e3..93102b2 100644
--- a/uimaj-examples/src/main/java/org/apache/uima/examples/xmi/Ecore2UimaTypeSystem.java
+++ b/uimaj-examples/src/main/java/org/apache/uima/examples/xmi/Ecore2UimaTypeSystem.java
@@ -318,10 +318,6 @@ public class Ecore2UimaTypeSystem {
     return type;
   }
 
-  /**
-   * @param attr -
-   * @return -
-   */
   private static FeatureDescription eStructuralFeature2UimaFeature(
           EStructuralFeature aStructuralFeature, Map aOptions) throws URISyntaxException {
     FeatureDescription feat = uimaFactory.createFeatureDescription();
@@ -418,10 +414,12 @@ public class Ecore2UimaTypeSystem {
       EPackage epackage = aEcoreType.getEPackage();
       if (epackage != null) {
         String uimaNamespace = namespaceUri2UimaNamespace(epackage.getNsURI());
-        if (uimaNamespace != null)
-          return uimaNamespace + '.' + aEcoreType.getName();
-        else
-          return aEcoreType.getName();
+        if (uimaNamespace != null) {
+            return uimaNamespace + '.' + aEcoreType.getName();
+        }
+        else {
+            return aEcoreType.getName();
+        }
       } else {
         return aEcoreType.getName();
       }
@@ -491,6 +489,8 @@ public class Ecore2UimaTypeSystem {
   /**
    * Main program. Takes two arguments: the filename of an input .ecore file and the filename of the
    * UIMA TypeSystem file to generate.
+   * @param args command line arguments.
+   * @throws Exception -
    */
   public static void main(String[] args) throws Exception {
     if (args.length != 2) {
diff --git a/uimaj-tools/src/main/java/org/apache/uima/tools/AnnotationViewerMain.java b/uimaj-tools/src/main/java/org/apache/uima/tools/AnnotationViewerMain.java
index d60fb78..3935688 100644
--- a/uimaj-tools/src/main/java/org/apache/uima/tools/AnnotationViewerMain.java
+++ b/uimaj-tools/src/main/java/org/apache/uima/tools/AnnotationViewerMain.java
@@ -192,7 +192,8 @@ public class AnnotationViewerMain extends JFrame {
 
     // Event Handlling of "Exit" Menu Item
     exitMenuItem.addActionListener(new ActionListener() {
-      public void actionPerformed(ActionEvent ae) {
+      @Override
+    public void actionPerformed(ActionEvent ae) {
         savePreferences();
         System.exit(0);
       }
@@ -200,14 +201,16 @@ public class AnnotationViewerMain extends JFrame {
 
     // Event Handlling of "About" Menu Item
     aboutMenuItem.addActionListener(new ActionListener() {
-      public void actionPerformed(ActionEvent ae) {
+      @Override
+    public void actionPerformed(ActionEvent ae) {
         aboutDialog.setVisible(true);
       }
     });
 
     // Event Handlling of "Help" Menu Item
     helpMenuItem.addActionListener(new ActionListener() {
-      public void actionPerformed(ActionEvent ae) {
+      @Override
+    public void actionPerformed(ActionEvent ae) {
         JOptionPane.showMessageDialog(AnnotationViewerMain.this, HELP_MESSAGE,
                 "Annotation Viewer Help", JOptionPane.PLAIN_MESSAGE);
       }
@@ -233,7 +236,8 @@ public class AnnotationViewerMain extends JFrame {
 
     // Event Handling of view Button
     viewButton.addActionListener(new ActionListener() {
-      public void actionPerformed(ActionEvent ee) {
+      @Override
+    public void actionPerformed(ActionEvent ee) {
         try {
           viewDocuments();
         } catch (Exception e) {
@@ -342,6 +346,8 @@ public class AnnotationViewerMain extends JFrame {
 
   /**
    * Gets the name of the style map file for the given AE or TypeSystem descriptor filename.
+   * @param aDescriptorFileName -
+   * @return -
    */
   public File getStyleMapFileName(String aDescriptorFileName) {
     String baseName;
@@ -358,7 +364,8 @@ public class AnnotationViewerMain extends JFrame {
     final AnnotationViewerMain frame = new AnnotationViewerMain();
 
     frame.addWindowListener(new WindowAdapter() {
-      public void windowClosing(WindowEvent e) {
+      @Override
+    public void windowClosing(WindowEvent e) {
         frame.savePreferences();
         System.exit(0);
       }
@@ -457,7 +464,8 @@ public class AnnotationViewerMain extends JFrame {
    * 
    * @see java.awt.Component#getPreferredSize()
    */
-  public Dimension getPreferredSize() {
+  @Override
+public Dimension getPreferredSize() {
     return new Dimension(640, 200);
   }  
 }
diff --git a/uimaj-tools/src/main/java/org/apache/uima/tools/RunAE.java b/uimaj-tools/src/main/java/org/apache/uima/tools/RunAE.java
index e2ab9ff..40c5cc1 100644
--- a/uimaj-tools/src/main/java/org/apache/uima/tools/RunAE.java
+++ b/uimaj-tools/src/main/java/org/apache/uima/tools/RunAE.java
@@ -105,6 +105,7 @@ public class RunAE implements StatusCallbackListener {
 
   /**
    * Constructor. Sets up and runs an Analysis Engine.
+   * @param args command line arguments.
    */
   public RunAE(String[] args) {
     try {
@@ -236,7 +237,8 @@ public class RunAE implements StatusCallbackListener {
   /**
    * @see org.apache.uima.collection.base_cpm.BaseStatusCallbackListener#initializationComplete()
    */
-  public void initializationComplete() {
+  @Override
+public void initializationComplete() {
   }
 
   /**
@@ -247,7 +249,8 @@ public class RunAE implements StatusCallbackListener {
    * @see org.apache.uima.collection.StatusCallbackListener#entityProcessComplete(org.apache.uima.cas.CAS,
    *      org.apache.uima.collection.EntityProcessStatus)
    */
-  public void entityProcessComplete(CAS aCas, EntityProcessStatus aStatus) {
+  @Override
+public void entityProcessComplete(CAS aCas, EntityProcessStatus aStatus) {
     if (aStatus.isException()) {
       Iterator iter = aStatus.getExceptions().iterator();
       while (iter.hasNext()) {
@@ -273,7 +276,8 @@ public class RunAE implements StatusCallbackListener {
   /**
    * @see org.apache.uima.collection.base_cpm.BaseStatusCallbackListener#aborted()
    */
-  public void aborted() {
+  @Override
+public void aborted() {
     System.out.println("Processing Aborted");
 
   }
@@ -281,13 +285,15 @@ public class RunAE implements StatusCallbackListener {
   /**
    * @see org.apache.uima.collection.base_cpm.BaseStatusCallbackListener#batchProcessComplete()
    */
-  public void batchProcessComplete() {
+  @Override
+public void batchProcessComplete() {
   }
 
   /**
    * @see org.apache.uima.collection.base_cpm.BaseStatusCallbackListener#collectionProcessComplete()
    */
-  public void collectionProcessComplete() {
+  @Override
+public void collectionProcessComplete() {
     // output performance stats
     if (statsLevel > 0) {
       AnalysisEnginePerformanceReports performanceReports = new AnalysisEnginePerformanceReports(
@@ -304,13 +310,15 @@ public class RunAE implements StatusCallbackListener {
   /**
    * @see org.apache.uima.collection.base_cpm.BaseStatusCallbackListener#paused()
    */
-  public void paused() {
+  @Override
+public void paused() {
   }
 
   /**
    * @see org.apache.uima.collection.base_cpm.BaseStatusCallbackListener#resumed()
    */
-  public void resumed() {
+  @Override
+public void resumed() {
   }
 
   /**
diff --git a/uimaj-tools/src/main/java/org/apache/uima/tools/cvd/MainFrame.java b/uimaj-tools/src/main/java/org/apache/uima/tools/cvd/MainFrame.java
index f652685..fb228e0 100644
--- a/uimaj-tools/src/main/java/org/apache/uima/tools/cvd/MainFrame.java
+++ b/uimaj-tools/src/main/java/org/apache/uima/tools/cvd/MainFrame.java
@@ -410,7 +410,7 @@ public class MainFrame extends JFrame {
 
   /**
    * Constructor for MainFrame.
-   * 
+   * @param iniFile -
    * @throws HeadlessException -
    */
   public MainFrame(File iniFile) {
@@ -1104,7 +1104,8 @@ public class MainFrame extends JFrame {
     this.showPerfReportItem = new JMenuItem("Performance report");
     this.showPerfReportItem.setEnabled(false);
     this.showPerfReportItem.addActionListener(new ActionListener() {
-      public void actionPerformed(ActionEvent e) {
+      @Override
+    public void actionPerformed(ActionEvent e) {
         if (MainFrame.this.lastRunProcessTrace == null) {
           MainFrame.this.showError("No performance report to show.");
         } else {
@@ -1171,7 +1172,8 @@ public class MainFrame extends JFrame {
     JMenuItem logViewer = new JMenuItem("View Log File", KeyEvent.VK_L);
     logViewer.addActionListener(new ActionListener() {
 
-      public void actionPerformed(ActionEvent event) {
+      @Override
+    public void actionPerformed(ActionEvent event) {
         LogFileViewer viewer = new LogFileViewer("Log file: "
             + MainFrame.this.logFile.getAbsolutePath());
         viewer.addWindowListener(new CloseLogViewHandler(MainFrame.this));
@@ -2191,7 +2193,7 @@ public class MainFrame extends JFrame {
    * window is being closed. Calling <code>setExitOnClose(false)</code> prevents that. It is then
    * the caller's task to shut down the JVM.
    * 
-   * @param exitOnClose
+   * @param exitOnClose exit-on-close behavior
    */
   public void setExitOnClose(boolean exitOnClose) {
     this.exitOnClose = exitOnClose;
diff --git a/uimaj-tools/src/main/java/org/apache/uima/tools/cvd/MultiAnnotViewerFrame.java b/uimaj-tools/src/main/java/org/apache/uima/tools/cvd/MultiAnnotViewerFrame.java
index 4d3b146..ac38247 100644
--- a/uimaj-tools/src/main/java/org/apache/uima/tools/cvd/MultiAnnotViewerFrame.java
+++ b/uimaj-tools/src/main/java/org/apache/uima/tools/cvd/MultiAnnotViewerFrame.java
@@ -40,35 +40,21 @@ public class MultiAnnotViewerFrame extends JFrame {
 
   private static final long serialVersionUID = -920372876117526451L;
 
-  /**
-   * @throws java.awt.HeadlessException
-   */
   public MultiAnnotViewerFrame() {
     super();
     // TODO Auto-generated constructor stub
   }
 
-  /**
-   * @param gc
-   */
   public MultiAnnotViewerFrame(GraphicsConfiguration gc) {
     super(gc);
     // TODO Auto-generated constructor stub
   }
 
-  /**
-   * @param title
-   * @throws java.awt.HeadlessException
-   */
   public MultiAnnotViewerFrame(String title) {
     super(title);
     // TODO Auto-generated constructor stub
   }
 
-  /**
-   * @param title
-   * @param gc
-   */
   public MultiAnnotViewerFrame(String title, GraphicsConfiguration gc) {
     super(title, gc);
     // TODO Auto-generated constructor stub
diff --git a/uimaj-tools/src/main/java/org/apache/uima/tools/cvd/control/IndexPopupListener.java b/uimaj-tools/src/main/java/org/apache/uima/tools/cvd/control/IndexPopupListener.java
index 01cd400..7002a27 100644
--- a/uimaj-tools/src/main/java/org/apache/uima/tools/cvd/control/IndexPopupListener.java
+++ b/uimaj-tools/src/main/java/org/apache/uima/tools/cvd/control/IndexPopupListener.java
@@ -37,17 +37,16 @@ public class IndexPopupListener extends MouseAdapter {
   
   private final MainFrame main;
 
-  /**
-   * @param frame
-   */
   public IndexPopupListener(MainFrame frame) {
     this.main = frame;
   }
 
+  @Override
   public void mousePressed(MouseEvent e) {
     maybeShowPopup(e);
   }
 
+  @Override
   public void mouseReleased(MouseEvent e) {
     maybeShowPopup(e);
   }
diff --git a/uimaj-tools/src/main/java/org/apache/uima/tools/cvd/control/ShowTypesystemHandler.java b/uimaj-tools/src/main/java/org/apache/uima/tools/cvd/control/ShowTypesystemHandler.java
index 0c732a8..7a51e39 100644
--- a/uimaj-tools/src/main/java/org/apache/uima/tools/cvd/control/ShowTypesystemHandler.java
+++ b/uimaj-tools/src/main/java/org/apache/uima/tools/cvd/control/ShowTypesystemHandler.java
@@ -33,9 +33,6 @@ public class ShowTypesystemHandler implements ActionListener {
   
   private final MainFrame main;
 
-  /**
-   * @param frame
-   */
   public ShowTypesystemHandler(MainFrame frame) {
     this.main = frame;
   }
@@ -43,6 +40,7 @@ public class ShowTypesystemHandler implements ActionListener {
   /**
    * @see java.awt.event.ActionListener#actionPerformed(ActionEvent)
    */
+  @Override
   public void actionPerformed(ActionEvent event) {
     if (this.main.getCas() == null) {
       return;
diff --git a/uimaj-tools/src/main/java/org/apache/uima/tools/cvd/tsview/FeatureTableModel.java b/uimaj-tools/src/main/java/org/apache/uima/tools/cvd/tsview/FeatureTableModel.java
index 0fd635a..425ba55 100644
--- a/uimaj-tools/src/main/java/org/apache/uima/tools/cvd/tsview/FeatureTableModel.java
+++ b/uimaj-tools/src/main/java/org/apache/uima/tools/cvd/tsview/FeatureTableModel.java
@@ -26,11 +26,6 @@ import javax.swing.table.AbstractTableModel;
 import org.apache.uima.cas.Feature;
 import org.apache.uima.cas.Type;
 
-/**
- * Insert comment for enclosing_type here.
- * 
- * 
- */
 public class FeatureTableModel extends AbstractTableModel {
 
   private static final long serialVersionUID = -6010925680514336742L;
@@ -43,9 +38,6 @@ public class FeatureTableModel extends AbstractTableModel {
     super();
   }
 
-  /**
-   * Constructor for FeatureTableModel.
-   */
   public FeatureTableModel(Type type) {
     super();
     this.type = type;
@@ -56,6 +48,7 @@ public class FeatureTableModel extends AbstractTableModel {
     fireTableDataChanged();
   }
 
+  @Override
   public String getColumnName(int i) {
     if (i < 0 || i >= columnHeaders.length) {
       return "";
@@ -66,6 +59,7 @@ public class FeatureTableModel extends AbstractTableModel {
   /**
    * @see javax.swing.table.TableModel#getRowCount()
    */
+  @Override
   public int getRowCount() {
     if (this.type == null) {
       return 0;
@@ -76,6 +70,7 @@ public class FeatureTableModel extends AbstractTableModel {
   /**
    * @see javax.swing.table.TableModel#getColumnCount()
    */
+  @Override
   public int getColumnCount() {
     return 3;
   }
@@ -83,6 +78,7 @@ public class FeatureTableModel extends AbstractTableModel {
   /**
    * @see javax.swing.table.TableModel#getValueAt(int, int)
    */
+  @Override
   public Object getValueAt(int rowIndex, int columnIndex) {
     if (this.type == null) {
       return null;
@@ -107,5 +103,4 @@ public class FeatureTableModel extends AbstractTableModel {
       }
     }
   }
-
 }
diff --git a/uimaj-tools/src/main/java/org/apache/uima/tools/cvd/tsview/MainFrame.java b/uimaj-tools/src/main/java/org/apache/uima/tools/cvd/tsview/MainFrame.java
index 86c20ab..bbba0fb 100644
--- a/uimaj-tools/src/main/java/org/apache/uima/tools/cvd/tsview/MainFrame.java
+++ b/uimaj-tools/src/main/java/org/apache/uima/tools/cvd/tsview/MainFrame.java
@@ -61,6 +61,7 @@ public class MainFrame extends JFrame {
     /**
      * @see javax.swing.event.TreeSelectionListener#valueChanged(javax.swing.event.TreeSelectionEvent)
      */
+    @Override
     public void valueChanged(TreeSelectionEvent event) {
       // System.out.println("");
       DefaultMutableTreeNode node = (DefaultMutableTreeNode) MainFrame.this.typeTree
@@ -96,33 +97,16 @@ public class MainFrame extends JFrame {
     init();
   }
 
-  /**
-   * Constructor for MainFrame.
-   * 
-   * @param gc
-   */
   public MainFrame(GraphicsConfiguration gc) {
     super(gc);
     init();
   }
 
-  /**
-   * Constructor for MainFrame.
-   * 
-   * @param title
-   * @throws HeadlessException -
-   */
   public MainFrame(String title) {
     super(title);
     init();
   }
 
-  /**
-   * Constructor for MainFrame.
-   * 
-   * @param title
-   * @param gc
-   */
   public MainFrame(String title, GraphicsConfiguration gc) {
     super(title, gc);
     init();
diff --git a/uimaj-tools/src/main/java/org/apache/uima/tools/docanalyzer/DocumentAnalyzer.java b/uimaj-tools/src/main/java/org/apache/uima/tools/docanalyzer/DocumentAnalyzer.java
index 28a5916..0276e7e 100644
--- a/uimaj-tools/src/main/java/org/apache/uima/tools/docanalyzer/DocumentAnalyzer.java
+++ b/uimaj-tools/src/main/java/org/apache/uima/tools/docanalyzer/DocumentAnalyzer.java
@@ -343,7 +343,8 @@ public class DocumentAnalyzer extends JFrame implements StatusCallbackListener,
     inputFileFormatComboBox.setEditable(false);
     inputFileFormatComboBox.setSelectedItem(prefsMed.getInputFileFormat());
     inputFileFormatComboBox.addActionListener(new ActionListener() {      
-      public void actionPerformed(ActionEvent e) {
+      @Override
+    public void actionPerformed(ActionEvent e) {
         String inputFileFormat = (String) inputFileFormatComboBox.getSelectedItem();
         lenientCheckbox.setEnabled(
             "xcas".equalsIgnoreCase(inputFileFormat) ||
@@ -424,7 +425,8 @@ public class DocumentAnalyzer extends JFrame implements StatusCallbackListener,
 
     // Event Handlling of "Exit" Menu Item
     exitMenuItem.addActionListener(new ActionListener() {
-      public void actionPerformed(ActionEvent evt) {
+      @Override
+    public void actionPerformed(ActionEvent evt) {
         savePreferences();
         System.exit(0);
       }
@@ -432,14 +434,16 @@ public class DocumentAnalyzer extends JFrame implements StatusCallbackListener,
 
     // Event Handlling of "About" Menu Item
     aboutMenuItem.addActionListener(new ActionListener() {
-      public void actionPerformed(ActionEvent evt) {
+      @Override
+    public void actionPerformed(ActionEvent evt) {
         aboutDialog.setVisible(true);
       }
     });
 
     // Event Handlling of "Help" Menu Item
     helpMenuItem.addActionListener(new ActionListener() {
-      public void actionPerformed(ActionEvent evt) {
+      @Override
+    public void actionPerformed(ActionEvent evt) {
         JOptionPane.showMessageDialog(DocumentAnalyzer.this, HELP_MESSAGE,
                 "Document Analyzer Help", JOptionPane.PLAIN_MESSAGE);
       }
@@ -481,7 +485,8 @@ public class DocumentAnalyzer extends JFrame implements StatusCallbackListener,
 
     // Event Handling of run Button
     runButton.addActionListener(new ActionListener() {
-      public void actionPerformed(ActionEvent ee) {
+      @Override
+    public void actionPerformed(ActionEvent ee) {
         interactive = false;
         savePreferences();
         analyzeDocuments(null); // JMP added arg
@@ -490,9 +495,11 @@ public class DocumentAnalyzer extends JFrame implements StatusCallbackListener,
 
     // Event Handling of interactive Button
     interButton.addActionListener(new ActionListener() {
-      public void actionPerformed(ActionEvent ee) {
-        if (outputFileSelector.getSelected().length() == 0)
-          displayError("Need to specify an output directory for temporary results.");
+      @Override
+    public void actionPerformed(ActionEvent ee) {
+        if (outputFileSelector.getSelected().length() == 0) {
+            displayError("Need to specify an output directory for temporary results.");
+        }
         else {
           interactive = true;
           savePreferences();
@@ -511,7 +518,8 @@ public class DocumentAnalyzer extends JFrame implements StatusCallbackListener,
     restorePreferences();
 
     progressTimer = new Timer(100, new ActionListener() {
-      public void actionPerformed(ActionEvent ee) {
+      @Override
+    public void actionPerformed(ActionEvent ee) {
         checkProgressMonitor();
       }
     });
@@ -539,7 +547,8 @@ public class DocumentAnalyzer extends JFrame implements StatusCallbackListener,
   }
 
   // View button clicked
-  public void actionPerformed(ActionEvent e) {
+  @Override
+public void actionPerformed(ActionEvent e) {
     savePreferences();
     try {
       aeSpecifierFile = new File(xmlFileSelector.getSelected());
@@ -564,18 +573,21 @@ public class DocumentAnalyzer extends JFrame implements StatusCallbackListener,
     outputFileSelected = outputFileSelector.getSelected();
 
     File inputDir = new File(inputFileSelector.getSelected());
-    if (outputFileSelector.getSelected().length() > 0)
-      outputDirectory = new File(outputFileSelector.getSelected());
-    else
-      outputDirectory = null;
+    if (outputFileSelector.getSelected().length() > 0) {
+        outputDirectory = new File(outputFileSelector.getSelected());
+    }
+    else {
+        outputDirectory = null;
+    }
 
     // reset file pointers in case of typed-in text. JMP
     String tempFileDir = null;
     if ((analysisText != null) && (outputDirectory != null)) {
       tempFileDir = outputFileSelector.getSelected() + "/interactive_temp";
       inputDir = new File(tempFileDir);
-      if (!inputDir.exists())
+      if (!inputDir.exists()) {
         inputDir.mkdirs();
+    }
       outputFileSelected = outputFileSelector.getSelected() + "/interactive_out";
       prefsMed.setOutputDirForInteractiveMode(outputFileSelected, outputFileSelector.getSelected());
       outputDirectory = new File(outputFileSelected);
@@ -700,7 +712,8 @@ public class DocumentAnalyzer extends JFrame implements StatusCallbackListener,
    * @see org.apache.uima.collection.StatusCallbackListener#entityProcessComplete(org.apache.uima.cas.CAS,
    *      org.apache.uima.collection.EntityProcessStatus)
    */
-  public void entityProcessComplete(CAS aCas, EntityProcessStatus aStatus) {
+  @Override
+public void entityProcessComplete(CAS aCas, EntityProcessStatus aStatus) {
     // if an error occurred, display error
     if (aStatus.isException()) {
       displayError((Throwable) aStatus.getExceptions().get(0));
@@ -718,7 +731,8 @@ public class DocumentAnalyzer extends JFrame implements StatusCallbackListener,
   /**
    * @see org.apache.uima.collection.base_cpm.BaseStatusCallbackListener#aborted()
    */
-  public void aborted() {
+  @Override
+public void aborted() {
     // close progress monitor
     if (progressMonitor != null) {
       progressMonitor.close();
@@ -733,13 +747,15 @@ public class DocumentAnalyzer extends JFrame implements StatusCallbackListener,
   /**
    * @see org.apache.uima.collection.base_cpm.BaseStatusCallbackListener#batchProcessComplete()
    */
-  public void batchProcessComplete() {
+  @Override
+public void batchProcessComplete() {
   }
 
   /**
    * @see org.apache.uima.collection.base_cpm.BaseStatusCallbackListener#collectionProcessComplete()
    */
-  public void collectionProcessComplete() {
+  @Override
+public void collectionProcessComplete() {
     // invoke ProcessingCompleteRunnable in Swing event handler thread
     // SwingUtilities.invokeLater(new ProcessingCompleteRunnable());
     // hide progress bar dialog if it is visible
@@ -763,19 +779,22 @@ public class DocumentAnalyzer extends JFrame implements StatusCallbackListener,
   /**
    * @see org.apache.uima.collection.base_cpm.BaseStatusCallbackListener#initializationComplete()
    */
-  public void initializationComplete() {
+  @Override
+public void initializationComplete() {
   }
 
   /**
    * @see org.apache.uima.collection.base_cpm.BaseStatusCallbackListener#paused()
    */
-  public void paused() {
+  @Override
+public void paused() {
   }
 
   /**
    * @see org.apache.uima.collection.base_cpm.BaseStatusCallbackListener#resumed()
    */
-  public void resumed() {
+  @Override
+public void resumed() {
   }
 
   /**
@@ -827,6 +846,8 @@ public class DocumentAnalyzer extends JFrame implements StatusCallbackListener,
    * Creates a CAS from an descriptor. Supports both local AE descriptors and remote service
    * specifiers. In the latter case the service is contacted to obtain its type system.
    * 
+   * @param aDescriptorFile -
+   * @return the new CAS
    * @throws ResourceInitializationException -
    * @throws InvalidXMLException -
    * @throws IOException -
@@ -1043,10 +1064,12 @@ public class DocumentAnalyzer extends JFrame implements StatusCallbackListener,
       displayFormatPanel.setLayout(new FlowLayout(FlowLayout.LEFT));
       displayFormatPanel.setBorder(BorderFactory.createEmptyBorder(0, 0, 0, 0));
 
-      if (generatedStyleMap)
+      if (generatedStyleMap) {
         javaViewerRB.setSelected(true);
-      else
+    }
+    else {
         javaViewerUCRB.setSelected(true);
+    }
 
       ButtonGroup displayFormatButtonGroup = new ButtonGroup();
       displayFormatButtonGroup.add(javaViewerRB);
@@ -1090,6 +1113,7 @@ public class DocumentAnalyzer extends JFrame implements StatusCallbackListener,
       mainPanel.add(southernPanel);
       // Event Handling of analyze Button
       analyzeButton.addActionListener(new ActionListener() {
+        @Override
         public void actionPerformed(ActionEvent ee) {
           analyzeTextArea();
         }
@@ -1098,6 +1122,7 @@ public class DocumentAnalyzer extends JFrame implements StatusCallbackListener,
       // event for the closeButton button
       closeButton.addActionListener(new ActionListener() {
 
+        @Override
         public void actionPerformed(ActionEvent evt) {
           setVisible(false);
           aiDialog.dispose();
diff --git a/uimaj-tools/src/main/java/org/apache/uima/tools/stylemap/StyleMapEditor.java b/uimaj-tools/src/main/java/org/apache/uima/tools/stylemap/StyleMapEditor.java
index 6c24202..1339f60 100644
--- a/uimaj-tools/src/main/java/org/apache/uima/tools/stylemap/StyleMapEditor.java
+++ b/uimaj-tools/src/main/java/org/apache/uima/tools/stylemap/StyleMapEditor.java
@@ -217,7 +217,8 @@ public class StyleMapEditor extends JDialog implements ActionListener {
 
     // Style Edit OK Button event handle
     okButton.addActionListener(new ActionListener() {
-      public void actionPerformed(ActionEvent evt) {
+      @Override
+    public void actionPerformed(ActionEvent evt) {
         Vector vn = new Vector();
         for (int row = 0; row < styleList.size(); row++) {
           StyleMapEntry e = (StyleMapEntry) styleList.get(row);
@@ -246,14 +247,16 @@ public class StyleMapEditor extends JDialog implements ActionListener {
           }
         }
 
-        if (buttonPress)
-          styleMapEditorDialog.setVisible(false);
+        if (buttonPress) {
+            styleMapEditorDialog.setVisible(false);
+        }
       }
     });
 
     // Style Edit Exit Button Event Handling
     cancelButton.addActionListener(new ActionListener() {
-      public void actionPerformed(ActionEvent evt) {
+      @Override
+    public void actionPerformed(ActionEvent evt) {
         buttonPress = false;
         styleMapEditorDialog.setVisible(false);
       }
@@ -264,7 +267,8 @@ public class StyleMapEditor extends JDialog implements ActionListener {
     pack();
   }
 
-  public Dimension getPreferredSize() {
+  @Override
+public Dimension getPreferredSize() {
     Dimension screenSize = getToolkit().getScreenSize();
     Dimension d = super.getPreferredSize();
     return new Dimension(screenSize.width, d.height);
@@ -273,6 +277,7 @@ public class StyleMapEditor extends JDialog implements ActionListener {
   }
 
   /**
+   * @param tae -
    * @deprecated use {@link #setAnalysisEngine(AnalysisEngineDescription)} instead.
    */
   @Deprecated
@@ -334,8 +339,9 @@ public class StyleMapEditor extends JDialog implements ActionListener {
     // the corresponding Color objects as values.
 
     // If stylemap XML is null, auto-generate it from metadata capabilities:
-    if (aStyleMapXml == null)
-      aStyleMapXml = AnnotationViewGenerator.autoGenerateStyleMap(aAnalysisEngineMetaData);
+    if (aStyleMapXml == null) {
+        aStyleMapXml = AnnotationViewGenerator.autoGenerateStyleMap(aAnalysisEngineMetaData);
+    }
 
     return parseStyleList(aStyleMapXml);
   }
@@ -364,7 +370,8 @@ public class StyleMapEditor extends JDialog implements ActionListener {
     return styleList;
   }
 
-  public void actionPerformed(ActionEvent evt) {
+  @Override
+public void actionPerformed(ActionEvent evt) {
     JComponent source = (JComponent) evt.getSource();
 
     if (source == moveRowUpButton) {
@@ -499,11 +506,13 @@ public class StyleMapEditor extends JDialog implements ActionListener {
 
         // Calculate adequate column widths:
         int typeNameWidth = fm.stringWidth(e.getAnnotationTypeName());
-        if (typeNameWidth > maxColumnWidths[StyleConstants.TYPE_NAME_COLUMN])
-          maxColumnWidths[StyleConstants.TYPE_NAME_COLUMN] = typeNameWidth;
+        if (typeNameWidth > maxColumnWidths[StyleConstants.TYPE_NAME_COLUMN]) {
+            maxColumnWidths[StyleConstants.TYPE_NAME_COLUMN] = typeNameWidth;
+        }
         int labelWidth = fm.stringWidth(e.getLabel().trim());
-        if (labelWidth > maxColumnWidths[StyleConstants.LABEL_COLUMN])
-          maxColumnWidths[StyleConstants.LABEL_COLUMN] = labelWidth;
+        if (labelWidth > maxColumnWidths[StyleConstants.LABEL_COLUMN]) {
+            maxColumnWidths[StyleConstants.LABEL_COLUMN] = labelWidth;
+        }
       }
 
       tableModel.set(data);
@@ -534,7 +543,8 @@ public class StyleMapEditor extends JDialog implements ActionListener {
     final JButton button = new JButton("") {
       private static final long serialVersionUID = 3955120051470642157L;
 
-      public void setText(String s) {
+      @Override
+    public void setText(String s) {
         // Button never shows text -- only color.
       }
     };
@@ -560,15 +570,18 @@ public class StyleMapEditor extends JDialog implements ActionListener {
     // colorChooser.setChooserPanels(preferredPanels);
 
     ActionListener okListener = new ActionListener() {
-      public void actionPerformed(ActionEvent e) {
+      @Override
+    public void actionPerformed(ActionEvent e) {
         colorEditor.currentColor = colorChooser.getColor();
         Color newColor = colorEditor.currentColor;
         StyleMapEntry entry = (StyleMapEntry) styleList.get(tbl.getSelectedRow());
         int column = tbl.getSelectedColumn();
-        if (column == StyleConstants.BG_COLUMN)
-          entry.setBackground(newColor);
-        else
-          entry.setForeground(newColor);
+        if (column == StyleConstants.BG_COLUMN) {
+            entry.setBackground(newColor);
+        }
+        else {
+            entry.setForeground(newColor);
+        }
 
         tbl.repaint();
       }
@@ -578,7 +591,8 @@ public class StyleMapEditor extends JDialog implements ActionListener {
 
     // Here's the code that brings up the dialog.
     button.addActionListener(new ActionListener() {
-      public void actionPerformed(ActionEvent e) {
+      @Override
+    public void actionPerformed(ActionEvent e) {
         button.setBackground(colorEditor.currentColor);
         colorChooser.setColor(colorEditor.currentColor);
         // Without the following line, the dialog comes up
@@ -618,12 +632,16 @@ public class StyleMapEditor extends JDialog implements ActionListener {
           newStyle.append("<label>");
           String label = e.getLabel();
           if (label != null) {
-            if ((label != null) && !label.equals(""))
-              newStyle.append(label);
-            else
-              newStyle.append(e.getAnnotationTypeName());
-          } else
+            if ((label != null) && !label.equals("")) {
+                newStyle.append(label);
+            }
+            else {
+                newStyle.append(e.getAnnotationTypeName());
+            }
+          }
+        else {
             newStyle.append(e.getAnnotationTypeName());
+        }
 
           newStyle.append("</label>\n");
           newStyle.append("<style>");
diff --git a/uimaj-tools/src/main/java/org/apache/uima/tools/util/gui/SpringUtilities.java b/uimaj-tools/src/main/java/org/apache/uima/tools/util/gui/SpringUtilities.java
index bc43c6d..dd1518d 100644
--- a/uimaj-tools/src/main/java/org/apache/uima/tools/util/gui/SpringUtilities.java
+++ b/uimaj-tools/src/main/java/org/apache/uima/tools/util/gui/SpringUtilities.java
@@ -33,6 +33,7 @@ public class SpringUtilities {
   /**
    * A debugging utility that prints to stdout the component's minimum, preferred, and maximum
    * sizes.
+   * @param c a component.
    */
   public static void printSizes(Component c) {
     System.out.println("minimumSize = " + c.getMinimumSize());