diff options
author | Andrew Ferguson | 2008-03-13 12:01:58 +0000 |
---|---|---|
committer | Andrew Ferguson | 2008-03-13 12:01:58 +0000 |
commit | f0495f62d0995a89ef62d3388342dacddc99dbb6 (patch) | |
tree | b6ac83ea605af0e9972e9410f5ae7aaed4d1aa8a /core/org.eclipse.cdt.core/templateengine/org | |
parent | 83b3470a9edbd79f66da26b11ba63d3d43f1b62f (diff) | |
download | org.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')
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(); } } |