You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@poi.apache.org by on...@apache.org on 2015/11/03 08:23:10 UTC
svn commit: r1712219 - in /poi/trunk/src: java/org/apache/poi/ss/util/
ooxml/testcases/org/apache/poi/xssf/usermodel/
testcases/org/apache/poi/ss/usermodel/ testcases/org/apache/poi/ss/util/
Author: onealj
Date: Tue Nov 3 07:23:09 2015
New Revision: 1712219
URL: http://svn.apache.org/viewvc?rev=1712219&view=rev
Log:
bug 58576: rename canComputeColumnWidht to canComputeColumnWidth
Modified:
poi/trunk/src/java/org/apache/poi/ss/util/SheetUtil.java
poi/trunk/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFFont.java
poi/trunk/src/testcases/org/apache/poi/ss/usermodel/BaseTestBugzillaIssues.java
poi/trunk/src/testcases/org/apache/poi/ss/util/TestSheetUtil.java
Modified: poi/trunk/src/java/org/apache/poi/ss/util/SheetUtil.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/java/org/apache/poi/ss/util/SheetUtil.java?rev=1712219&r1=1712218&r2=1712219&view=diff
==============================================================================
--- poi/trunk/src/java/org/apache/poi/ss/util/SheetUtil.java (original)
+++ poi/trunk/src/java/org/apache/poi/ss/util/SheetUtil.java Tue Nov 3 07:23:09 2015
@@ -275,7 +275,7 @@ public class SheetUtil {
* @param font The Font that is used in the Cell
* @return true if computing the size for this Font will succeed, false otherwise
*/
- public static boolean canComputeColumnWidht(Font font) {
+ public static boolean canComputeColumnWidth(Font font) {
// not sure what is the best value sample-here, only "1" did not work on some platforms...
AttributedString str = new AttributedString("1w");
copyAttributes(font, str, 0, "1w".length());
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=1712219&r1=1712218&r2=1712219&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 Tue Nov 3 07:23:09 2015
@@ -299,7 +299,7 @@ public final class TestXSSFFont extends
Workbook wb = new XSSFWorkbook();
// cannot check on result because on some machines we get back false here!
- SheetUtil.canComputeColumnWidht(wb.getFontAt((short)0));
+ SheetUtil.canComputeColumnWidth(wb.getFontAt((short)0));
wb.close();
}
@@ -310,6 +310,6 @@ public final class TestXSSFFont extends
font.setFontName("some non existing font name");
// Even with invalid fonts we still get back useful data most of the time...
- SheetUtil.canComputeColumnWidht(font);
+ SheetUtil.canComputeColumnWidth(font);
}
}
Modified: poi/trunk/src/testcases/org/apache/poi/ss/usermodel/BaseTestBugzillaIssues.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/testcases/org/apache/poi/ss/usermodel/BaseTestBugzillaIssues.java?rev=1712219&r1=1712218&r2=1712219&view=diff
==============================================================================
--- poi/trunk/src/testcases/org/apache/poi/ss/usermodel/BaseTestBugzillaIssues.java (original)
+++ poi/trunk/src/testcases/org/apache/poi/ss/usermodel/BaseTestBugzillaIssues.java Tue Nov 3 07:23:09 2015
@@ -381,7 +381,7 @@ public abstract class BaseTestBugzillaIs
// autoSize will fail if required fonts are not installed, skip this test then
Font font = wb.getFontAt(cell0.getCellStyle().getFontIndex());
Assume.assumeTrue("Cannot verify autoSizeColumn() because the necessary Fonts are not installed on this machine: " + font,
- SheetUtil.canComputeColumnWidht(font));
+ SheetUtil.canComputeColumnWidth(font));
assertEquals("Expecting no indentation in this test",
0, cell0.getCellStyle().getIndention());
@@ -399,7 +399,7 @@ public abstract class BaseTestBugzillaIs
double widthBeforeCol = SheetUtil.getColumnWidth(sheet, 0, false);
String info = widthManual + "/" + widthBeforeCell + "/" + widthBeforeCol + "/" +
- SheetUtil.canComputeColumnWidht(font) + "/" + computeCellWidthFixed(font, "1") + "/" + computeCellWidthFixed(font, "w") + "/" +
+ SheetUtil.canComputeColumnWidth(font) + "/" + computeCellWidthFixed(font, "1") + "/" + computeCellWidthFixed(font, "w") + "/" +
computeCellWidthFixed(font, "1w") + "/" + computeCellWidthFixed(font, "0000") + "/" + computeCellWidthFixed(font, longValue);
assertTrue("Expected to have cell width > 0 when computing manually, but had " + info, widthManual > 0);
assertTrue("Expected to have cell width > 0 BEFORE auto-size, but had " + info, widthBeforeCell > 0);
Modified: poi/trunk/src/testcases/org/apache/poi/ss/util/TestSheetUtil.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/testcases/org/apache/poi/ss/util/TestSheetUtil.java?rev=1712219&r1=1712218&r2=1712219&view=diff
==============================================================================
--- poi/trunk/src/testcases/org/apache/poi/ss/util/TestSheetUtil.java (original)
+++ poi/trunk/src/testcases/org/apache/poi/ss/util/TestSheetUtil.java Tue Nov 3 07:23:09 2015
@@ -72,7 +72,7 @@ public final class TestSheetUtil extends
Workbook wb = new HSSFWorkbook();
// cannot check on result because on some machines we get back false here!
- SheetUtil.canComputeColumnWidht(wb.getFontAt((short)0));
+ SheetUtil.canComputeColumnWidth(wb.getFontAt((short)0));
wb.close();
}
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@poi.apache.org
For additional commands, e-mail: commits-help@poi.apache.org