Skip to main content
aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAndrew Ferguson2008-03-13 12:01:58 +0000
committerAndrew Ferguson2008-03-13 12:01:58 +0000
commitf0495f62d0995a89ef62d3388342dacddc99dbb6 (patch)
treeb6ac83ea605af0e9972e9410f5ae7aaed4d1aa8a /core/org.eclipse.cdt.core/templateengine/org
parent83b3470a9edbd79f66da26b11ba63d3d43f1b62f (diff)
downloadorg.eclipse.cdt-f0495f62d0995a89ef62d3388342dacddc99dbb6.tar.gz
org.eclipse.cdt-f0495f62d0995a89ef62d3388342dacddc99dbb6.tar.xz
org.eclipse.cdt-f0495f62d0995a89ef62d3388342dacddc99dbb6.zip
clean warnings
Diffstat (limited to 'core/org.eclipse.cdt.core/templateengine/org')
-rw-r--r--core/org.eclipse.cdt.core/templateengine/org/eclipse/cdt/core/templateengine/TemplateDescriptor.java24
-rw-r--r--core/org.eclipse.cdt.core/templateengine/org/eclipse/cdt/core/templateengine/process/ConditionalProcessGroup.java30
-rw-r--r--core/org.eclipse.cdt.core/templateengine/org/eclipse/cdt/core/templateengine/process/Process.java10
-rw-r--r--core/org.eclipse.cdt.core/templateengine/org/eclipse/cdt/core/templateengine/process/ProcessArgument.java26
4 files changed, 45 insertions, 45 deletions
diff --git a/core/org.eclipse.cdt.core/templateengine/org/eclipse/cdt/core/templateengine/TemplateDescriptor.java b/core/org.eclipse.cdt.core/templateengine/org/eclipse/cdt/core/templateengine/TemplateDescriptor.java
index 9f31d6de309..38f17237106 100644
--- a/core/org.eclipse.cdt.core/templateengine/org/eclipse/cdt/core/templateengine/TemplateDescriptor.java
+++ b/core/org.eclipse.cdt.core/templateengine/org/eclipse/cdt/core/templateengine/TemplateDescriptor.java
@@ -71,10 +71,10 @@ public class TemplateDescriptor {
*
* @return default values with keys
*/
- public Map<String, String> getTemplateDefaults(Element rootElement) {
+ public Map<String, String> getTemplateDefaults(Element element) {
Map<String, String> templateDefaults = new HashMap<String, String>();
Element propertyGroupElement;
- List<Element> children = TemplateEngine.getChildrenOfElement(rootElement);
+ List<Element> children = TemplateEngine.getChildrenOfElement(element);
for (int i = 0, l = children.size(); i < l; i++) {
propertyGroupElement = children.get(i);
if (isNestedElement(propertyGroupElement)) {
@@ -134,24 +134,24 @@ public class TemplateDescriptor {
}
/**
- * This mehtod is to get the complex property-group from template descriptor
+ * This method is to get the complex property-group from template descriptor
* root element. complex means a property-group contains other
* property-group(s)
*
- * @param rootElement
+ * @param element
* root element of type JDOM Element
- * @return porperty-group root element of type JDOM Element
+ * @return property-group root element of type JDOM Element
*/
- public Element getRootPropertyGroup(Element rootElement) {
- if (rootElement != null) {
- String rootElementName = rootElement.getNodeName();
- if (rootElementName.equalsIgnoreCase(PROPERTY_GROUP) && isNestedElement(rootElement)) {
- return rootElement;
+ public Element getRootPropertyGroup(Element element) {
+ if (element != null) {
+ String rootElementName = element.getNodeName();
+ if (rootElementName.equalsIgnoreCase(PROPERTY_GROUP) && isNestedElement(element)) {
+ return element;
}
- return rootElement;
+ return element;
} else {
String nextPGElementName = null;
- List<Element> propertyGroupList = TemplateEngine.getChildrenOfElement(rootElement);
+ List<Element> propertyGroupList = TemplateEngine.getChildrenOfElement(element);
for (int i = 0, l = propertyGroupList.size(); i < l; i++) {
Element nextPGElement = propertyGroupList.get(i);
if (isNestedElement(nextPGElement))
diff --git a/core/org.eclipse.cdt.core/templateengine/org/eclipse/cdt/core/templateengine/process/ConditionalProcessGroup.java b/core/org.eclipse.cdt.core/templateengine/org/eclipse/cdt/core/templateengine/process/ConditionalProcessGroup.java
index 9a13faaeceb..f802d1ac626 100644
--- a/core/org.eclipse.cdt.core/templateengine/org/eclipse/cdt/core/templateengine/process/ConditionalProcessGroup.java
+++ b/core/org.eclipse.cdt.core/templateengine/org/eclipse/cdt/core/templateengine/process/ConditionalProcessGroup.java
@@ -43,7 +43,7 @@ public class ConditionalProcessGroup {
private String conditionString;
private String lValue;
private String rValue;
- private Operator op;
+ private Operator operator;
private List<Process> processes;
private String id;
@@ -80,13 +80,13 @@ public class ConditionalProcessGroup {
} else {
int op = conditionString.indexOf(ProcessHelper.EQUALS);
if (op != -1) {
- this.op = Operator.EQUALS;
+ this.operator = Operator.EQUALS;
lValue = conditionString.substring(0, op);
rValue = conditionString.substring(op + ProcessHelper.EQUALS.length());
} else {
op = conditionString.indexOf(ProcessHelper.NOT_EQUALS);
if (op != -1) {
- this.op = Operator.NOT_EQUALS;
+ this.operator = Operator.NOT_EQUALS;
lValue = conditionString.substring(0, op);
rValue = conditionString.substring(op + ProcessHelper.NOT_EQUALS.length());
}//else an unsupported operation where this condition is ignored.
@@ -125,17 +125,17 @@ public class ConditionalProcessGroup {
/**
* Creates the Process from the process Elements.
- * @param template
+ * @param templateCore
* @param processElements
*/
- private void createProcessObjects(TemplateCore template, List<Element> processElements) {
- this.template = template;
+ private void createProcessObjects(TemplateCore templateCore, List<Element> processElements) {
+ this.template = templateCore;
this.processes = new ArrayList<Process>(processElements.size());
for (int j = 0, l = processElements.size(); j < l; j++) {
Element processElem = processElements.get(j);
if (processElem.getNodeName().equals(TemplateDescriptor.PROCESS)) {
String processId = id + "--> Process " + (j + 1) + " (" + processElem.getAttribute(Process.ELEM_TYPE) + ")"; //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$
- processes.add(new Process(template, processElem, processId));
+ processes.add(new Process(templateCore, processElem, processId));
}
}
}
@@ -188,16 +188,16 @@ public class ConditionalProcessGroup {
return false;
}
Map<String, String> valueStore = template.getValueStore();
- String lValue = this.lValue;
- String rValue = this.rValue;
+ String processedLValue= lValue;
+ String processedRValue= rValue;
for(String value : macros) {
- lValue = lValue.replaceAll(ProcessHelper.START_PATTERN + value + ProcessHelper.END_PATTERN, valueStore.get(value));
- rValue = rValue.replaceAll(ProcessHelper.START_PATTERN + value + ProcessHelper.END_PATTERN, valueStore.get(value));
+ processedLValue = processedLValue.replaceAll(ProcessHelper.START_PATTERN + value + ProcessHelper.END_PATTERN, valueStore.get(value));
+ processedRValue = processedRValue.replaceAll(ProcessHelper.START_PATTERN + value + ProcessHelper.END_PATTERN, valueStore.get(value));
}
- if(op.equals(Operator.EQUALS)) {
- return lValue.equals(rValue);
- } else if(op.equals(Operator.NOT_EQUALS)) {
- return !lValue.equals(rValue);
+ if(operator.equals(Operator.EQUALS)) {
+ return processedLValue.equals(processedRValue);
+ } else if(operator.equals(Operator.NOT_EQUALS)) {
+ return !processedLValue.equals(processedRValue);
} else {
return false;
}
diff --git a/core/org.eclipse.cdt.core/templateengine/org/eclipse/cdt/core/templateengine/process/Process.java b/core/org.eclipse.cdt.core/templateengine/org/eclipse/cdt/core/templateengine/process/Process.java
index d9d612d9ae9..ceaf533d41c 100644
--- a/core/org.eclipse.cdt.core/templateengine/org/eclipse/cdt/core/templateengine/process/Process.java
+++ b/core/org.eclipse.cdt.core/templateengine/org/eclipse/cdt/core/templateengine/process/Process.java
@@ -57,10 +57,10 @@ public class Process {
/**
* This method build the necessary Arguments for the process
- * @param template
+ * @param templateCore
* @param element
*/
- private void buildArgs(TemplateCore template, Element element) {
+ private void buildArgs(TemplateCore templateCore, Element element) {
List<Element> children = TemplateEngine.getChildrenOfElement(element);
ProcessParameter[] params = processRunner.getProcessParameters();
List<ProcessArgument> list = new ArrayList<ProcessArgument>(params.length);
@@ -70,14 +70,14 @@ public class Process {
boolean childrenRemain = childIndex < children.size();
Element child = (childrenRemain ? children.get(childIndex) : null);
if (param.isExternal() && (!childrenRemain || !param.getName().equals(child.getAttribute(ProcessArgument.ELEM_NAME)))) {
- list.add(new ProcessArgument(template, param));
+ list.add(new ProcessArgument(templateCore, param));
} else if (childrenRemain) {
- list.add(new ProcessArgument(template, child));
+ list.add(new ProcessArgument(templateCore, child));
childIndex++;
}
}
while (childIndex < children.size()) {
- list.add(new ProcessArgument(template, children.get(childIndex++)));
+ list.add(new ProcessArgument(templateCore, children.get(childIndex++)));
}
args = list.toArray(new ProcessArgument[list.size()]);
}
diff --git a/core/org.eclipse.cdt.core/templateengine/org/eclipse/cdt/core/templateengine/process/ProcessArgument.java b/core/org.eclipse.cdt.core/templateengine/org/eclipse/cdt/core/templateengine/process/ProcessArgument.java
index 987410f1935..48a654fe845 100644
--- a/core/org.eclipse.cdt.core/templateengine/org/eclipse/cdt/core/templateengine/process/ProcessArgument.java
+++ b/core/org.eclipse.cdt.core/templateengine/org/eclipse/cdt/core/templateengine/process/ProcessArgument.java
@@ -207,12 +207,12 @@ public class ProcessArgument {
case ProcessParameter.COMPLEX_ARRAY:
params = param.getComplexChildren();
for(int i=0; i<complexValueArray.length; i++) {
- ProcessArgument[] complexValue = complexValueArray[i];
- if (params.length != complexValue.length) {
+ ProcessArgument[] cValue = complexValueArray[i];
+ if (params.length != cValue.length) {
return false;
}
- for (int j = 0; j < complexValue.length; j++) {
- if (!complexValue[j].isOfParameterType(params[j])) {
+ for (int j = 0; j < cValue.length; j++) {
+ if (!cValue[j].isOfParameterType(params[j])) {
return false;
}
}
@@ -256,9 +256,9 @@ public class ProcessArgument {
return true;
case ProcessParameter.COMPLEX_ARRAY:
for(int i=0; i<complexValueArray.length; i++) {
- ProcessArgument[] complexValue =complexValueArray[i];
- for(int j=0; j<complexValue.length; j++) {
- ProcessArgument arg = complexValue[j];
+ ProcessArgument[] cValue =complexValueArray[i];
+ for(int j=0; j<cValue.length; j++) {
+ ProcessArgument arg = cValue[j];
if (!arg.areAllMacrosExpandable()) {
return false;
}
@@ -297,9 +297,9 @@ public class ProcessArgument {
return null;
case ProcessParameter.COMPLEX_ARRAY:
for(int i=0; i<complexValueArray.length; i++) {
- ProcessArgument[] complexValue =complexValueArray[i];
- for(int j=0; j<complexValue.length; j++) {
- ProcessArgument arg = complexValue[j];
+ ProcessArgument[] cValue =complexValueArray[i];
+ for(int j=0; j<cValue.length; j++) {
+ ProcessArgument arg = cValue[j];
if ((macro = arg.getFirstNonExpandableMacro()) != null) {
return macro;
}
@@ -351,9 +351,9 @@ public class ProcessArgument {
break;
case ProcessParameter.COMPLEX_ARRAY:
for(int i=0; i<complexValueArray.length; i++) {
- ProcessArgument[] complexValue =complexValueArray[i];
- for(int j=0; j<complexValue.length; j++) {
- ProcessArgument arg = complexValue[j];
+ ProcessArgument[] cValue =complexValueArray[i];
+ for(int j=0; j<cValue.length; j++) {
+ ProcessArgument arg = cValue[j];
arg.resolve();
}
}

Back to the top