fix more warnings

core/org.eclipse.cdt.core/templateengine/org/eclipse/cdt/core/templateengine/TemplateDescriptor.java

@@ -110,10 +110,8 @@ public class TemplateDescriptor {
 
 	/**
 	 * This method is to get the list of property-group elements from template
-	 * descriptor root element
+	 * descriptor root element.
 	 * 
-	 * @param aRootElement
-	 *            root element of type JDOM Element
 	 * @return list of property-group elements
 	 */
 	public List<Element> getPropertyGroupList() {
@@ -156,7 +154,7 @@ public class TemplateDescriptor {
 				Element nextPGElement = propertyGroupList.get(i);
 				if (isNestedElement(nextPGElement))
 					nextPGElementName = nextPGElement.getNodeName();
-				if (nextPGElementName.equalsIgnoreCase(PROPERTY_GROUP) && isNestedElement(nextPGElement)) {
+				if (PROPERTY_GROUP.equalsIgnoreCase(nextPGElementName) && isNestedElement(nextPGElement)) {
 					return nextPGElement;
 				}
 			}

core/org.eclipse.cdt.core/templateengine/org/eclipse/cdt/core/templateengine/process/ConditionalProcessGroup.java

@@ -253,8 +253,7 @@ public class ConditionalProcessGroup {
 	}
 
 	/**
-	 * Returns All Macros as a Set.
-	 * @return Set, contains macros
+	 * @return the union of all macros used in the child processes
 	 */
 	public Set<String> getAllMacros() {
 		Set<String> set = null;

core/org.eclipse.cdt.core/templateengine/org/eclipse/cdt/core/templateengine/process/ProcessHelper.java

@@ -43,10 +43,8 @@ public class ProcessHelper {
 	/**
 	 * This method is to append the given contents into a file.
 	 * 
-	 * @param fileContents,
-	 *            contents which are appended to the file.
-	 * @param toFile,
-	 *            a file to append contents.
+	 * @param fileContents contents which are appended to the file.
+	 * @param toFile a file to append contents.
 	 * @throws IOException,
 	 *             exception while writing contents into a file
      * 
@@ -62,17 +60,18 @@ public class ProcessHelper {
 				raf.skipBytes((int) raf.length());
 				raf.writeBytes(fileContents);
 			} finally {
+				if(raf != null) {
 					raf.close();
 				}
 			}
 		}
+	}
 
 	/**
 	 * This method returns a vector of all replace marker strings. (e.g.,
 	 * $(item), vector contains 'item' as one item. , ) is the end pattern.
 	 * 
-	 * @param str,
-	 *            A given string possibly containing markers.
+	 * @param str A given string possibly containing markers.
 	 * @return the set of names occurring within markers
      * @since 4.0
 	 */
@@ -172,11 +171,10 @@ public class ProcessHelper {
 
 
 	/**
-	 * Returns the Macro Value after expanding the Macros.
 	 * @param string
 	 * @param macros
 	 * @param valueStore
-	 * @return
+	 * @return the Macro Value after expanding the Macros.
      * 
      * @since 4.0
 	 */
@@ -192,10 +190,8 @@ public class ProcessHelper {
 	}
 
 	/**
-	 * Construct and return the replacement markers 
-	 * after adding the patterns to the macro.
 	 * @param macro
-	 * @return
+	 * @return the replacement marker string
      * 
      * @since 4.0
 	 */

core/org.eclipse.cdt.core/templateengine/org/eclipse/cdt/core/templateengine/process/TemplateProcessHandler.java

@@ -71,8 +71,7 @@ public class TemplateProcessHandler {
 	}
 
 	/**
-	 * Returns all macros
-	 * @return
+	 * @return the union of all macros used in this template's process groups
 	 */
 	public Set<String> getAllMacros() {
 		Set<String> set = new HashSet<String>();