Browse Source

fix several findbugs SF_SWITCH_NO_DEFAULT warnings

git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1738782 13f79535-47bb-0310-9956-ffa450edef68
tags/REL_3_15_BETA2
Javen O'Neal 8 years ago
parent
commit
2f703ce676

+ 2
- 0
src/java/org/apache/poi/hssf/usermodel/HSSFPatriarch.java View File

case HSSFWorkbook.PICTURE_TYPE_PICT: case HSSFWorkbook.PICTURE_TYPE_PICT:
ftCf.setFlags(FtCfSubRecord.BITMAP_BIT); ftCf.setFlags(FtCfSubRecord.BITMAP_BIT);
break; break;
default:
throw new IllegalStateException("Invalid picture type: " + pictData.getFormat());
} }
obj.addSubRecord(ftCf); obj.addSubRecord(ftCf);
// FtPioGrbit (pictFlags) // FtPioGrbit (pictFlags)

+ 2
- 0
src/java/org/apache/poi/hssf/usermodel/HSSFShapeFactory.java View File

case EscherTextboxRecord.RECORD_ID: case EscherTextboxRecord.RECORD_ID:
txtRecord = (TextObjectRecord) shapeToObj.get(record); txtRecord = (TextObjectRecord) shapeToObj.get(record);
break; break;
default:
break;
} }
} }
if (isEmbeddedObject(objRecord)) { if (isEmbeddedObject(objRecord)) {

+ 2
- 0
src/java/org/apache/poi/hssf/usermodel/HSSFShapeGroup.java View File

case EscherChildAnchorRecord.RECORD_ID: case EscherChildAnchorRecord.RECORD_ID:
anchor = new HSSFChildAnchor((EscherChildAnchorRecord) ch); anchor = new HSSFChildAnchor((EscherChildAnchorRecord) ch);
break; break;
default:
break;
} }
} }
} }

+ 10
- 2
src/java/org/apache/poi/hssf/usermodel/HSSFWorkbook.java View File

* @param format The format of the picture. One of <code>PICTURE_TYPE_*</code> * @param format The format of the picture. One of <code>PICTURE_TYPE_*</code>
* *
* @return the index to this picture (1 based). * @return the index to this picture (1 based).
* @see #PICTURE_TYPE_WMF
* @see #PICTURE_TYPE_EMF
* @see #PICTURE_TYPE_PICT
* @see #PICTURE_TYPE_PNG
* @see #PICTURE_TYPE_JPEG
* @see #PICTURE_TYPE_DIB
*/ */
@Override @Override
public int addPicture(byte[] pictureData, int format) public int addPicture(byte[] pictureData, int format)
case PICTURE_TYPE_PNG: case PICTURE_TYPE_PNG:
blipRecord.setOptions(HSSFPictureData.MSOBI_PNG); blipRecord.setOptions(HSSFPictureData.MSOBI_PNG);
break; break;
case HSSFWorkbook.PICTURE_TYPE_JPEG:
case PICTURE_TYPE_JPEG:
blipRecord.setOptions(HSSFPictureData.MSOBI_JPEG); blipRecord.setOptions(HSSFPictureData.MSOBI_JPEG);
break; break;
case HSSFWorkbook.PICTURE_TYPE_DIB:
case PICTURE_TYPE_DIB:
blipRecord.setOptions(HSSFPictureData.MSOBI_DIB); blipRecord.setOptions(HSSFPictureData.MSOBI_DIB);
break; break;
default:
throw new IllegalStateException("Unexpected picture format: " + format);
} }


EscherBSERecord r = new EscherBSERecord(); EscherBSERecord r = new EscherBSERecord();

+ 2
- 1
src/java/org/apache/poi/ss/formula/functions/FinanceFunction.java View File

} }
return evaluate(srcRowIndex, srcColumnIndex, args[0], args[1], args[2], arg3, arg4); return evaluate(srcRowIndex, srcColumnIndex, args[0], args[1], args[2], arg3, arg4);
} }
default:
return ErrorEval.VALUE_INVALID;
} }
return ErrorEval.VALUE_INVALID;
} }


protected double evaluate(double[] ds) throws EvaluationException { protected double evaluate(double[] ds) throws EvaluationException {

+ 6
- 0
src/java/org/apache/poi/ss/formula/functions/Offset.java View File

if(!(args[4] instanceof MissingArgEval)) { if(!(args[4] instanceof MissingArgEval)) {
width = evaluateIntArg(args[4], srcCellRow, srcCellCol); width = evaluateIntArg(args[4], srcCellRow, srcCellCol);
} }
// fall-through to pick up height
case 4: case 4:
if(!(args[3] instanceof MissingArgEval)) { if(!(args[3] instanceof MissingArgEval)) {
height = evaluateIntArg(args[3], srcCellRow, srcCellCol); height = evaluateIntArg(args[3], srcCellRow, srcCellCol);
} }
break;
//case 3:
// nothing to do
default:
break;
} }
// Zero height or width raises #REF! error // Zero height or width raises #REF! error
if(height == 0 || width == 0) { if(height == 0 || width == 0) {

+ 7
- 4
src/java/org/apache/poi/util/HexDump.java View File

if (Character.isISOControl(charB)) return '.'; if (Character.isISOControl(charB)) return '.';
switch (charB) { switch (charB) {
// printable, but not compilable with current compiler encoding
case 0xFF: case 0xDD:
charB = '.';
break;
// printable, but not compilable with current compiler encoding
case 0xFF:
case 0xDD:
charB = '.';
break;
default:
break;
} }
return charB; return charB;
} }

+ 2
- 0
src/ooxml/java/org/apache/poi/xslf/usermodel/XSLFFreeformShape.java View File

numPoints++; numPoints++;
ctPath.addNewClose(); ctPath.addNewClose();
break; break;
default:
throw new IllegalStateException("Unrecognized path segment type: " + type);
} }
it.next(); it.next();
} }

+ 3
- 1
src/ooxml/java/org/apache/poi/xssf/streaming/SheetDataWriter.java View File

_out.write("<v>" + nval + "</v>"); _out.write("<v>" + nval + "</v>");
} }
break; break;
default:
break;
} }
break; break;
} }
break; break;
} }
default: { default: {
throw new RuntimeException("Huh?");
throw new IllegalStateException("Invalid cell type: " + cellType);
} }
} }
_out.write("</c>"); _out.write("</c>");

+ 2
- 0
src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFFont.java View File

case Font.SS_SUPER: case Font.SS_SUPER:
offsetProperty.setVal(STVerticalAlignRun.SUPERSCRIPT); offsetProperty.setVal(STVerticalAlignRun.SUPERSCRIPT);
break; break;
default:
throw new IllegalStateException("Invalid type offset: " + offset);
} }
} }
} }

+ 4
- 0
src/scratchpad/src/org/apache/poi/hwpf/model/FileInformationBlock.java View File

case 0x0112: case 0x0112:
assertCbRgFcLcb( "0x0112", 0x00B7, "0x00B7", _cbRgFcLcb ); assertCbRgFcLcb( "0x0112", 0x00B7, "0x00B7", _cbRgFcLcb );
break; break;
default:
throw new IllegalStateException("Invalid file format version number: " + getNFib());
} }
} }


case 0x0112: case 0x0112:
assertCswNew( "0x0112", 0x0005, "0x0005", _cswNew ); assertCswNew( "0x0112", 0x0005, "0x0005", _cswNew );
break; break;
default:
throw new IllegalStateException("Invalid file format version number: " + getNFib());
} }
} }



Loading…
Cancel
Save