Some Eclipse warnings removed

git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1516980 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
Dominik Stadler 2013-08-23 18:54:05 +00:00
parent 689aa10390
commit f8f998b80b
4 changed files with 13 additions and 9 deletions

View File

@ -49,8 +49,7 @@ public class FormulaRenderer {
} }
Stack<String> stack = new Stack<String>(); Stack<String> stack = new Stack<String>();
for (int i=0 ; i < ptgs.length; i++) { for (Ptg ptg : ptgs) {
Ptg ptg = ptgs[i];
// TODO - what about MemNoMemPtg? // TODO - what about MemNoMemPtg?
if(ptg instanceof MemAreaPtg || ptg instanceof MemFuncPtg || ptg instanceof MemErrPtg) { if(ptg instanceof MemAreaPtg || ptg instanceof MemFuncPtg || ptg instanceof MemErrPtg) {
// marks the start of a list of area expressions which will be naturally combined // marks the start of a list of area expressions which will be naturally combined

View File

@ -17,10 +17,10 @@
package org.apache.poi.ss.formula.ptg; package org.apache.poi.ss.formula.ptg;
import org.apache.poi.ss.util.AreaReference;
import org.apache.poi.ss.formula.ExternSheetReferenceToken; import org.apache.poi.ss.formula.ExternSheetReferenceToken;
import org.apache.poi.ss.formula.FormulaRenderingWorkbook; import org.apache.poi.ss.formula.FormulaRenderingWorkbook;
import org.apache.poi.ss.formula.WorkbookDependentFormula; import org.apache.poi.ss.formula.WorkbookDependentFormula;
import org.apache.poi.ss.util.AreaReference;
import org.apache.poi.util.LittleEndianInput; import org.apache.poi.util.LittleEndianInput;
import org.apache.poi.util.LittleEndianOutput; import org.apache.poi.util.LittleEndianOutput;
@ -61,6 +61,7 @@ public final class Area3DPtg extends AreaPtgBase implements WorkbookDependentFor
setExternSheetIndex(externIdx); setExternSheetIndex(externIdx);
} }
@Override
public String toString() { public String toString() {
StringBuffer sb = new StringBuffer(); StringBuffer sb = new StringBuffer();
sb.append(getClass().getName()); sb.append(getClass().getName());
@ -72,12 +73,14 @@ public final class Area3DPtg extends AreaPtgBase implements WorkbookDependentFor
return sb.toString(); return sb.toString();
} }
@Override
public void write(LittleEndianOutput out) { public void write(LittleEndianOutput out) {
out.writeByte(sid + getPtgClass()); out.writeByte(sid + getPtgClass());
out.writeShort(field_1_index_extern_sheet); out.writeShort(field_1_index_extern_sheet);
writeCoordinates(out); writeCoordinates(out);
} }
@Override
public int getSize() { public int getSize() {
return SIZE; return SIZE;
} }
@ -99,6 +102,7 @@ public final class Area3DPtg extends AreaPtgBase implements WorkbookDependentFor
public String toFormulaString(FormulaRenderingWorkbook book) { public String toFormulaString(FormulaRenderingWorkbook book) {
return ExternSheetNameResolver.prependSheetName(book, field_1_index_extern_sheet, formatReferenceAsString()); return ExternSheetNameResolver.prependSheetName(book, field_1_index_extern_sheet, formatReferenceAsString());
} }
@Override
public String toFormulaString() { public String toFormulaString() {
throw new RuntimeException("3D references need a workbook to determine formula text"); throw new RuntimeException("3D references need a workbook to determine formula text");
} }

View File

@ -39,6 +39,7 @@ public final class AreaPtg extends Area2DPtgBase {
public AreaPtg(AreaReference areaRef) { public AreaPtg(AreaReference areaRef) {
super(areaRef); super(areaRef);
} }
@Override
protected byte getSid() { protected byte getSid() {
return sid; return sid;
} }

View File

@ -17,10 +17,6 @@
package org.apache.poi.xssf.usermodel; package org.apache.poi.xssf.usermodel;
import org.apache.poi.ss.formula.functions.FreeRefFunction;
import org.apache.poi.ss.formula.ptg.NamePtg;
import org.apache.poi.ss.formula.ptg.NameXPtg;
import org.apache.poi.ss.formula.ptg.Ptg;
import org.apache.poi.ss.SpreadsheetVersion; import org.apache.poi.ss.SpreadsheetVersion;
import org.apache.poi.ss.formula.EvaluationCell; import org.apache.poi.ss.formula.EvaluationCell;
import org.apache.poi.ss.formula.EvaluationName; import org.apache.poi.ss.formula.EvaluationName;
@ -30,8 +26,12 @@ import org.apache.poi.ss.formula.FormulaParser;
import org.apache.poi.ss.formula.FormulaParsingWorkbook; import org.apache.poi.ss.formula.FormulaParsingWorkbook;
import org.apache.poi.ss.formula.FormulaRenderingWorkbook; import org.apache.poi.ss.formula.FormulaRenderingWorkbook;
import org.apache.poi.ss.formula.FormulaType; import org.apache.poi.ss.formula.FormulaType;
import org.apache.poi.ss.formula.udf.UDFFinder; import org.apache.poi.ss.formula.functions.FreeRefFunction;
import org.apache.poi.ss.formula.ptg.NamePtg;
import org.apache.poi.ss.formula.ptg.NameXPtg;
import org.apache.poi.ss.formula.ptg.Ptg;
import org.apache.poi.ss.formula.udf.IndexedUDFFinder; import org.apache.poi.ss.formula.udf.IndexedUDFFinder;
import org.apache.poi.ss.formula.udf.UDFFinder;
import org.openxmlformats.schemas.spreadsheetml.x2006.main.CTDefinedName; import org.openxmlformats.schemas.spreadsheetml.x2006.main.CTDefinedName;
/** /**
@ -67,7 +67,7 @@ public final class XSSFEvaluationWorkbook implements FormulaRenderingWorkbook, E
* @return the external sheet index of the sheet with the given internal * @return the external sheet index of the sheet with the given internal
* index. Used by some of the more obscure formula and named range things. * index. Used by some of the more obscure formula and named range things.
* Fairly easy on XSSF (we think...) since the internal and external * Fairly easy on XSSF (we think...) since the internal and external
* indicies are the same * indices are the same
*/ */
private int convertToExternalSheetIndex(int sheetIndex) { private int convertToExternalSheetIndex(int sheetIndex) {
return sheetIndex; return sheetIndex;