You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@poi.apache.org by fa...@apache.org on 2018/03/03 01:48:47 UTC
svn commit: r1825749 - in /poi/trunk/src: java/org/apache/poi/ss/util/
ooxml/java/org/apache/poi/xssf/streaming/
ooxml/java/org/apache/poi/xssf/usermodel/
ooxml/testcases/org/apache/poi/xssf/usermodel/
Author: fanningpj
Date: Sat Mar 3 01:48:46 2018
New Revision: 1825749
URL: http://svn.apache.org/viewvc?rev=1825749&view=rev
Log:
remove some casts to short
Modified:
poi/trunk/src/java/org/apache/poi/ss/util/PropertyTemplate.java
poi/trunk/src/java/org/apache/poi/ss/util/RegionUtil.java
poi/trunk/src/ooxml/java/org/apache/poi/xssf/streaming/SXSSFCell.java
poi/trunk/src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFFontFormatting.java
poi/trunk/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFBugs.java
poi/trunk/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFColor.java
poi/trunk/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFFont.java
Modified: poi/trunk/src/java/org/apache/poi/ss/util/PropertyTemplate.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/java/org/apache/poi/ss/util/PropertyTemplate.java?rev=1825749&r1=1825748&r2=1825749&view=diff
==============================================================================
--- poi/trunk/src/java/org/apache/poi/ss/util/PropertyTemplate.java (original)
+++ poi/trunk/src/java/org/apache/poi/ss/util/PropertyTemplate.java Sat Mar 3 01:48:46 2018
@@ -958,8 +958,8 @@ public final class PropertyTemplate {
* @return short value, or 0 if not a short
*/
private static short getShort(Object value) {
- if (value instanceof Short) {
- return ((Short) value).shortValue();
+ if (value instanceof Number) {
+ return ((Number) value).shortValue();
}
return 0;
}
Modified: poi/trunk/src/java/org/apache/poi/ss/util/RegionUtil.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/java/org/apache/poi/ss/util/RegionUtil.java?rev=1825749&r1=1825748&r2=1825749&view=diff
==============================================================================
--- poi/trunk/src/java/org/apache/poi/ss/util/RegionUtil.java (original)
+++ poi/trunk/src/java/org/apache/poi/ss/util/RegionUtil.java Sat Mar 3 01:48:46 2018
@@ -42,7 +42,7 @@ public final class RegionUtil {
public CellPropertySetter(String propertyName, int value) {
_propertyName = propertyName;
- _propertyValue = Short.valueOf((short) value);
+ _propertyValue = Integer.valueOf(value);
}
public CellPropertySetter(String propertyName, BorderStyle value) {
_propertyName = propertyName;
Modified: poi/trunk/src/ooxml/java/org/apache/poi/xssf/streaming/SXSSFCell.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/ooxml/java/org/apache/poi/xssf/streaming/SXSSFCell.java?rev=1825749&r1=1825748&r2=1825749&view=diff
==============================================================================
--- poi/trunk/src/ooxml/java/org/apache/poi/xssf/streaming/SXSSFCell.java (original)
+++ poi/trunk/src/ooxml/java/org/apache/poi/xssf/streaming/SXSSFCell.java Sat Mar 3 01:48:46 2018
@@ -613,7 +613,7 @@ public class SXSSFCell implements Cell {
{
if(_style == null){
SXSSFWorkbook wb = (SXSSFWorkbook)getRow().getSheet().getWorkbook();
- return wb.getCellStyleAt((short)0);
+ return wb.getCellStyleAt(0);
} else {
return _style;
}
Modified: poi/trunk/src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFFontFormatting.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFFontFormatting.java?rev=1825749&r1=1825748&r2=1825749&view=diff
==============================================================================
--- poi/trunk/src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFFontFormatting.java (original)
+++ poi/trunk/src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFFontFormatting.java Sat Mar 3 01:48:46 2018
@@ -138,7 +138,7 @@ public class XSSFFontFormatting implemen
if(_font.sizeOfSzArray() == 0) return -1;
CTFontSize sz = _font.getSzArray(0);
- return (short)(20*sz.getVal());
+ return (int)(20*sz.getVal());
}
/**
Modified: poi/trunk/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFBugs.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFBugs.java?rev=1825749&r1=1825748&r2=1825749&view=diff
==============================================================================
--- poi/trunk/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFBugs.java (original)
+++ poi/trunk/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFBugs.java Sat Mar 3 01:48:46 2018
@@ -876,7 +876,7 @@ public final class TestXSSFBugs extends
font1.setColor((short) 20);
Font font2 = wb1.createFont();
font2.setColor(Font.COLOR_RED);
- Font font3 = wb1.getFontAt((short) 0);
+ Font font3 = wb1.getFontAt(0);
XSSFRow row = sheet.createRow(2);
XSSFCell cell = row.createCell(2);
@@ -1750,9 +1750,9 @@ public final class TestXSSFBugs extends
// run some method on the font to verify if it is "disconnected" already
//for(short i = 0;i < 256;i++)
{
- Font font = wb.getFontAt((short) 0);
+ Font font = wb.getFontAt(0);
if (font instanceof XSSFFont) {
- XSSFFont xfont = (XSSFFont) wb.getFontAt((short) 0);
+ XSSFFont xfont = (XSSFFont) wb.getFontAt(0);
CTFontImpl ctFont = (CTFontImpl) xfont.getCTFont();
assertEquals(0, ctFont.sizeOfBArray());
}
Modified: poi/trunk/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFColor.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFColor.java?rev=1825749&r1=1825748&r2=1825749&view=diff
==============================================================================
--- poi/trunk/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFColor.java (original)
+++ poi/trunk/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFColor.java Sat Mar 3 01:48:46 2018
@@ -35,7 +35,7 @@ public final class TestXSSFColor {
XSSFWorkbook wb = XSSFTestDataSamples.openSampleWorkbook("48779.xlsx");
// Check the CTColor is as expected
- XSSFColor indexed = wb.getCellStyleAt((short)1).getFillBackgroundXSSFColor();
+ XSSFColor indexed = wb.getCellStyleAt(1).getFillBackgroundXSSFColor();
assertEquals(true, indexed.getCTColor().isSetIndexed());
assertEquals(64, indexed.getCTColor().getIndexed());
assertEquals(false, indexed.getCTColor().isSetRgb());
Modified: poi/trunk/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFFont.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFFont.java?rev=1825749&r1=1825748&r2=1825749&view=diff
==============================================================================
--- poi/trunk/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFFont.java (original)
+++ poi/trunk/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFFont.java Sat Mar 3 01:48:46 2018
@@ -323,7 +323,7 @@ public final class TestXSSFFont extends
Workbook wb = new XSSFWorkbook();
// cannot check on result because on some machines we get back false here!
- SheetUtil.canComputeColumnWidth(wb.getFontAt((short)0));
+ SheetUtil.canComputeColumnWidth(wb.getFontAt(0));
wb.close();
}
@@ -354,16 +354,16 @@ public final class TestXSSFFont extends
assertEquals(1, wb.getNumberOfFonts());
- XSSFFont f1 = wb.getFontAt((short) 0);
+ XSSFFont f1 = wb.getFontAt(0);
assertFalse(f1.getBold());
// Check that asking for the same font
// multiple times gives you the same thing.
// Otherwise, our tests wouldn't work!
- assertSame(wb.getFontAt((short) 0), wb.getFontAt((short) 0));
+ assertSame(wb.getFontAt(0), wb.getFontAt(0));
assertEquals(
- wb.getFontAt((short) 0),
- wb.getFontAt((short) 0)
+ wb.getFontAt(0),
+ wb.getFontAt(0)
);
// Look for a new font we have
@@ -384,8 +384,8 @@ public final class TestXSSFFont extends
XSSFFont nf = wb.createFont();
assertEquals(2, wb.getNumberOfFonts());
- assertEquals(1, nf.getIndex());
- assertEquals(nf, wb.getFontAt((short) 1));
+ assertEquals(1, nf.getIndexAsInt());
+ assertEquals(nf, wb.getFontAt(1));
nf.setBold(false);
nf.setColor(IndexedColors.INDIGO.getIndex());
@@ -397,12 +397,12 @@ public final class TestXSSFFont extends
nf.setUnderline((byte) 2);
assertEquals(2, wb.getNumberOfFonts());
- assertEquals(nf, wb.getFontAt((short) 1));
+ assertEquals(nf, wb.getFontAt(1));
assertTrue(
- wb.getFontAt((short) 0)
+ wb.getFontAt(0)
!=
- wb.getFontAt((short) 1)
+ wb.getFontAt(1)
);
// Find it now
@@ -426,7 +426,7 @@ public final class TestXSSFFont extends
assertNotNull(font);
assertEquals(
1,
- font.getIndex()
+ font.getIndexAsInt()
);
assertEquals(nf,
wb.findFont(
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@poi.apache.org
For additional commands, e-mail: commits-help@poi.apache.org