Author: fanningpj
Date: Thu Dec 30 21:35:45 2021
New Revision: 1896547
URL: http://svn.apache.org/viewvc?rev=1896547&view=rev
Log:
use isEmpty()
Modified:
poi/trunk/poi-excelant/src/main/java/org/apache/poi/ss/excelant/ExcelAntTest.java
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xddf/usermodel/text/XDDFTextParagraph.java
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xslf/usermodel/XSLFTextParagraph.java
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/binary/XSSFBSharedStringsTable.java
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/binary/XSSFBSheetHandler.java
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/eventusermodel/ReadOnlySharedStringsTable.java
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/model/StylesTable.java
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFSheet.java
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFTextParagraph.java
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFWorkbook.java
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xwpf/usermodel/XWPFEndnote.java
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xwpf/usermodel/XWPFFootnote.java
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xwpf/usermodel/XWPFParagraph.java
poi/trunk/poi-ooxml/src/test/java/org/apache/poi/xssf/extractor/TestXSSFExportToXML.java
poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hslf/usermodel/HSLFSlideShowImpl.java
poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hsmf/MAPIMessage.java
poi/trunk/poi/src/main/java/org/apache/poi/hssf/usermodel/HSSFChart.java
poi/trunk/poi/src/main/java/org/apache/poi/hssf/usermodel/HSSFSheet.java
poi/trunk/poi/src/main/java/org/apache/poi/poifs/property/DirectoryProperty.java
poi/trunk/poi/src/main/java/org/apache/poi/ss/format/CellNumberFormatter.java
poi/trunk/poi/src/main/java/org/apache/poi/ss/format/CellNumberPartHandler.java
poi/trunk/poi/src/test/java/org/apache/poi/poifs/filesystem/TestPOIFSFileSystem.java
Modified:
poi/trunk/poi-excelant/src/main/java/org/apache/poi/ss/excelant/ExcelAntTest.java
URL:
http://svn.apache.org/viewvc/poi/trunk/poi-excelant/src/main/java/org/apache/poi/ss/excelant/ExcelAntTest.java?rev=1896547&r1=1896546&r2=1896547&view=diff
==============================================================================
---
poi/trunk/poi-excelant/src/main/java/org/apache/poi/ss/excelant/ExcelAntTest.java
(original)
+++
poi/trunk/poi-excelant/src/main/java/org/apache/poi/ss/excelant/ExcelAntTest.java
Thu Dec 30 21:35:45 2021
@@ -191,7 +191,7 @@ public class ExcelAntTest extends Task{
" of " + testCount + " evaluations failed to " +
"evaluate correctly.",
Project.MSG_ERR );
- if(showFailureDetail && failureMessages.size() > 0 ) {
+ if(showFailureDetail && !failureMessages.isEmpty()) {
for (String failureMessage : failureMessages) {
log(failureMessage, Project.MSG_ERR);
}
Modified:
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xddf/usermodel/text/XDDFTextParagraph.java
URL:
http://svn.apache.org/viewvc/poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xddf/usermodel/text/XDDFTextParagraph.java?rev=1896547&r1=1896546&r2=1896547&view=diff
==============================================================================
---
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xddf/usermodel/text/XDDFTextParagraph.java
(original)
+++
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xddf/usermodel/text/XDDFTextParagraph.java
Thu Dec 30 21:35:45 2021
@@ -80,7 +80,7 @@ public class XDDFTextParagraph implement
public void setText(String text) {
// keep the properties of current last run
XmlObject existing = null;
- if (_runs.size() > 0) {
+ if (!_runs.isEmpty()) {
existing = _runs.get(_runs.size() - 1).getProperties().copy();
}
// remove all runs
Modified:
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xslf/usermodel/XSLFTextParagraph.java
URL:
http://svn.apache.org/viewvc/poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xslf/usermodel/XSLFTextParagraph.java?rev=1896547&r1=1896546&r2=1896547&view=diff
==============================================================================
---
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xslf/usermodel/XSLFTextParagraph.java
(original)
+++
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xslf/usermodel/XSLFTextParagraph.java
Thu Dec 30 21:35:45 2021
@@ -134,7 +134,7 @@ public class XSLFTextParagraph implement
public XSLFTextRun addLineBreak(){
XSLFLineBreak run = new XSLFLineBreak(_p.addNewBr(), this);
CTTextCharacterProperties brProps = run.getRPr(true);
- if(_runs.size() > 0){
+ if (!_runs.isEmpty()) {
// by default line break has the font size of the last text run
CTTextCharacterProperties prevRun = _runs.get(_runs.size() -
1).getRPr(true);
brProps.set(prevRun);
Modified:
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/binary/XSSFBSharedStringsTable.java
URL:
http://svn.apache.org/viewvc/poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/binary/XSSFBSharedStringsTable.java?rev=1896547&r1=1896546&r2=1896547&view=diff
==============================================================================
---
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/binary/XSSFBSharedStringsTable.java
(original)
+++
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/binary/XSSFBSharedStringsTable.java
Thu Dec 30 21:35:45 2021
@@ -66,7 +66,7 @@ public class XSSFBSharedStringsTable imp
pkg.getPartsByContentType(XSSFBRelation.SHARED_STRINGS_BINARY.getContentType());
// Some workbooks have no shared strings table.
- if (parts.size() > 0) {
+ if (!parts.isEmpty()) {
PackagePart sstPart = parts.get(0);
try (InputStream stream = sstPart.getInputStream()) {
readFrom(stream);
Modified:
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/binary/XSSFBSheetHandler.java
URL:
http://svn.apache.org/viewvc/poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/binary/XSSFBSheetHandler.java?rev=1896547&r1=1896546&r2=1896547&view=diff
==============================================================================
---
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/binary/XSSFBSheetHandler.java
(original)
+++
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/binary/XSSFBSheetHandler.java
Thu Dec 30 21:35:45 2021
@@ -226,7 +226,7 @@ public class XSSFBSheetHandler extends X
private void outputHeaderFooter(XSSFBHeaderFooter headerFooter) {
String text = headerFooter.getString();
- if (text != null && text.trim().length() > 0) {
+ if (text != null && !text.trim().isEmpty()) {
handler.headerFooter(text, headerFooter.isHeader(),
headerFooter.getHeaderFooterTypeLabel());
}
}
@@ -238,7 +238,7 @@ public class XSSFBSheetHandler extends X
return;
}
Queue<CellAddress> queue = comments.getAddresses();
- while (queue.size() > 0) {
+ while (!queue.isEmpty()) {
CellAddress cellAddress = queue.peek();
if (cellAddress.getRow() == currentRow && cellAddress.getColumn()
< colNum) {
cellAddress = queue.remove();
@@ -261,7 +261,7 @@ public class XSSFBSheetHandler extends X
}
Queue<CellAddress> queue = comments.getAddresses();
int lastInterpolatedRow = -1;
- while (queue.size() > 0) {
+ while (!queue.isEmpty()) {
CellAddress cellAddress = queue.peek();
if (currentRow == CHECK_ALL_ROWS || cellAddress.getRow() <
currentRow) {
cellAddress = queue.remove();
Modified:
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/eventusermodel/ReadOnlySharedStringsTable.java
URL:
http://svn.apache.org/viewvc/poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/eventusermodel/ReadOnlySharedStringsTable.java?rev=1896547&r1=1896546&r2=1896547&view=diff
==============================================================================
---
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/eventusermodel/ReadOnlySharedStringsTable.java
(original)
+++
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/eventusermodel/ReadOnlySharedStringsTable.java
Thu Dec 30 21:35:45 2021
@@ -129,7 +129,7 @@ public class ReadOnlySharedStringsTable
pkg.getPartsByContentType(XSSFRelation.SHARED_STRINGS.getContentType());
// Some workbooks have no shared strings table.
- if (parts.size() > 0) {
+ if (!parts.isEmpty()) {
PackagePart sstPart = parts.get(0);
try (InputStream stream = sstPart.getInputStream()) {
readFrom(stream);
Modified:
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/model/StylesTable.java
URL:
http://svn.apache.org/viewvc/poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/model/StylesTable.java?rev=1896547&r1=1896546&r2=1896547&view=diff
==============================================================================
---
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/model/StylesTable.java
(original)
+++
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/model/StylesTable.java
Thu Dec 30 21:35:45 2021
@@ -693,7 +693,7 @@ public class StylesTable extends POIXMLD
styleSheet.setBorders(ctBorders);
// Xfs
- if(xfs.size() > 0) {
+ if (!xfs.isEmpty()) {
CTCellXfs ctXfs = styleSheet.getCellXfs();
if (ctXfs == null) {
ctXfs = CTCellXfs.Factory.newInstance();
@@ -706,7 +706,7 @@ public class StylesTable extends POIXMLD
}
// Style xfs
- if(styleXfs.size() > 0) {
+ if (!styleXfs.isEmpty()) {
CTCellStyleXfs ctSXfs = styleSheet.getCellStyleXfs();
if (ctSXfs == null) {
ctSXfs = CTCellStyleXfs.Factory.newInstance();
@@ -719,7 +719,7 @@ public class StylesTable extends POIXMLD
}
// Style dxfs
- if(dxfs.size() > 0) {
+ if (!dxfs.isEmpty()) {
CTDxfs ctDxfs = styleSheet.getDxfs();
if (ctDxfs == null) {
ctDxfs = CTDxfs.Factory.newInstance();
Modified:
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFSheet.java
URL:
http://svn.apache.org/viewvc/poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFSheet.java?rev=1896547&r1=1896546&r2=1896547&view=diff
==============================================================================
---
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFSheet.java
(original)
+++
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFSheet.java
Thu Dec 30 21:35:45 2021
@@ -3632,7 +3632,7 @@ public class XSSFSheet extends POIXMLDoc
}
// Now re-generate our CTHyperlinks, if needed
- if(hyperlinks.size() > 0) {
+ if (!hyperlinks.isEmpty()) {
if(worksheet.getHyperlinks() == null) {
worksheet.addNewHyperlinks();
}
Modified:
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFTextParagraph.java
URL:
http://svn.apache.org/viewvc/poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFTextParagraph.java?rev=1896547&r1=1896546&r2=1896547&view=diff
==============================================================================
---
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFTextParagraph.java
(original)
+++
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFTextParagraph.java
Thu Dec 30 21:35:45 2021
@@ -112,7 +112,7 @@ public class XSSFTextParagraph implement
public XSSFTextRun addLineBreak(){
CTTextLineBreak br = _p.addNewBr();
CTTextCharacterProperties brProps = br.addNewRPr();
- if(_runs.size() > 0){
+ if (!_runs.isEmpty()) {
// by default line break has the font size of the last text run
CTTextCharacterProperties prevRun = _runs.get(_runs.size() -
1).getRPr();
brProps.set(prevRun);
Modified:
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFWorkbook.java
URL:
http://svn.apache.org/viewvc/poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFWorkbook.java?rev=1896547&r1=1896546&r2=1896547&view=diff
==============================================================================
---
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFWorkbook.java
(original)
+++
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFWorkbook.java
Thu Dec 30 21:35:45 2021
@@ -1741,7 +1741,7 @@ public class XSSFWorkbook extends POIXML
*/
private void saveNamedRanges(){
// Named ranges
- if(namedRanges.size() > 0) {
+ if (!namedRanges.isEmpty()) {
CTDefinedNames names = CTDefinedNames.Factory.newInstance();
CTDefinedName[] nr = new CTDefinedName[namedRanges.size()];
int i = 0;
Modified:
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xwpf/usermodel/XWPFEndnote.java
URL:
http://svn.apache.org/viewvc/poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xwpf/usermodel/XWPFEndnote.java?rev=1896547&r1=1896546&r2=1896547&view=diff
==============================================================================
---
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xwpf/usermodel/XWPFEndnote.java
(original)
+++
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xwpf/usermodel/XWPFEndnote.java
Thu Dec 30 21:35:45 2021
@@ -68,7 +68,7 @@ public class XWPFEndnote extends XWPFAbs
public void ensureFootnoteRef(XWPFParagraph p) {
XWPFRun r = null;
- if (p.getRuns().size() > 0) {
+ if (!p.runsIsEmpty()) {
r = p.getRuns().get(0);
}
if (r == null) {
Modified:
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xwpf/usermodel/XWPFFootnote.java
URL:
http://svn.apache.org/viewvc/poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xwpf/usermodel/XWPFFootnote.java?rev=1896547&r1=1896546&r2=1896547&view=diff
==============================================================================
---
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xwpf/usermodel/XWPFFootnote.java
(original)
+++
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xwpf/usermodel/XWPFFootnote.java
Thu Dec 30 21:35:45 2021
@@ -63,7 +63,7 @@ public class XWPFFootnote extends XWPFAb
public void ensureFootnoteRef(XWPFParagraph p) {
XWPFRun r = null;
- if (p.getRuns().size() > 0) {
+ if (!p.runsIsEmpty()) {
r = p.getRuns().get(0);
}
if (r == null) {
Modified:
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xwpf/usermodel/XWPFParagraph.java
URL:
http://svn.apache.org/viewvc/poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xwpf/usermodel/XWPFParagraph.java?rev=1896547&r1=1896546&r2=1896547&view=diff
==============================================================================
---
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xwpf/usermodel/XWPFParagraph.java
(original)
+++
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xwpf/usermodel/XWPFParagraph.java
Thu Dec 30 21:35:45 2021
@@ -180,6 +180,10 @@ public class XWPFParagraph implements IB
return Collections.unmodifiableList(runs);
}
+ public boolean runsIsEmpty() {
+ return runs.isEmpty();
+ }
+
/**
* Return literal runs and sdt/content control objects.
*/
Modified:
poi/trunk/poi-ooxml/src/test/java/org/apache/poi/xssf/extractor/TestXSSFExportToXML.java
URL:
http://svn.apache.org/viewvc/poi/trunk/poi-ooxml/src/test/java/org/apache/poi/xssf/extractor/TestXSSFExportToXML.java?rev=1896547&r1=1896546&r2=1896547&view=diff
==============================================================================
---
poi/trunk/poi-ooxml/src/test/java/org/apache/poi/xssf/extractor/TestXSSFExportToXML.java
(original)
+++
poi/trunk/poi-ooxml/src/test/java/org/apache/poi/xssf/extractor/TestXSSFExportToXML.java
Thu Dec 30 21:35:45 2021
@@ -641,7 +641,7 @@ public final class TestXSSFExportToXML {
void testBug59026() throws Exception {
try (XSSFWorkbook wb =
XSSFTestDataSamples.openSampleWorkbook("59026.xlsx")) {
Collection<XSSFMap> mappings = wb.getCustomXMLMappings();
- assertTrue(mappings.size() > 0);
+ assertFalse(mappings.isEmpty());
for (XSSFMap map : mappings) {
XSSFExportToXml exporter = new XSSFExportToXml(map);
Modified:
poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hslf/usermodel/HSLFSlideShowImpl.java
URL:
http://svn.apache.org/viewvc/poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hslf/usermodel/HSLFSlideShowImpl.java?rev=1896547&r1=1896546&r2=1896547&view=diff
==============================================================================
---
poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hslf/usermodel/HSLFSlideShowImpl.java
(original)
+++
poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hslf/usermodel/HSLFSlideShowImpl.java
Thu Dec 30 21:35:45 2021
@@ -906,7 +906,7 @@ public final class HSLFSlideShowImpl ext
// Add the new picture in
int offset = 0;
- if (_pictures.size() > 0) {
+ if (!_pictures.isEmpty()) {
HSLFPictureData prev = _pictures.get(_pictures.size() - 1);
offset = prev.getOffset() + prev.getBseSize();
}
Modified:
poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hsmf/MAPIMessage.java
URL:
http://svn.apache.org/viewvc/poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hsmf/MAPIMessage.java?rev=1896547&r1=1896546&r2=1896547&view=diff
==============================================================================
--- poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hsmf/MAPIMessage.java
(original)
+++ poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hsmf/MAPIMessage.java
Thu Dec 30 21:35:45 2021
@@ -226,7 +226,7 @@ public class MAPIMessage extends POIRead
ByteChunk htmlBodyBinaryChunk = mainChunks.getHtmlBodyChunkBinary();
if (htmlBodyBinaryChunk != null) {
List<PropertyValue> cpid =
mainChunks.getProperties().get(MAPIProperty.INTERNET_CPID);
- if (cpid != null && cpid.size() > 0) {
+ if (cpid != null && !cpid.isEmpty()) {
int codepage = ((LongPropertyValue) cpid.get(0)).getValue();
try {
String encoding = CodePageUtil.codepageToEncoding(codepage,
true);
@@ -423,7 +423,7 @@ public class MAPIMessage extends POIRead
// General codepage: Message codepage property.
//
List<PropertyValue> val =
mainChunks.getProperties().get(MAPIProperty.MESSAGE_CODEPAGE);
- if (val != null && val.size() > 0) {
+ if (val != null && !val.isEmpty()) {
int codepage = ((LongPropertyValue) val.get(0)).getValue();
try {
String encoding = CodePageUtil.codepageToEncoding(codepage, true);
@@ -437,7 +437,7 @@ public class MAPIMessage extends POIRead
//
if (generalcodepage == null) {
val = mainChunks.getProperties().get(MAPIProperty.MESSAGE_LOCALE_ID);
- if (val != null && val.size() > 0) {
+ if (val != null && !val.isEmpty()) {
int lcid = ((LongPropertyValue) val.get(0)).getValue();
int codepage = LocaleUtil.getDefaultCodePageFromLCID(lcid);
try {
@@ -476,7 +476,7 @@ public class MAPIMessage extends POIRead
// UTF-8 is ignored for text body. This seems to be a special Outlook
behavior.
//
val = mainChunks.getProperties().get(MAPIProperty.INTERNET_CPID);
- if (val != null && val.size() > 0) {
+ if (val != null && !val.isEmpty()) {
int codepage = ((LongPropertyValue) val.get(0)).getValue();
try {
String encoding = CodePageUtil.codepageToEncoding(codepage, true);
@@ -644,7 +644,7 @@ public class MAPIMessage extends POIRead
MAPIProperty.CREATION_TIME
}) {
List<PropertyValue> val = mainChunks.getProperties().get(prop);
- if (val != null && val.size() > 0) {
+ if (val != null && !val.isEmpty()) {
return ((TimePropertyValue)val.get(0)).getValue();
}
}
Modified:
poi/trunk/poi/src/main/java/org/apache/poi/hssf/usermodel/HSSFChart.java
URL:
http://svn.apache.org/viewvc/poi/trunk/poi/src/main/java/org/apache/poi/hssf/usermodel/HSSFChart.java?rev=1896547&r1=1896546&r2=1896547&view=diff
==============================================================================
--- poi/trunk/poi/src/main/java/org/apache/poi/hssf/usermodel/HSSFChart.java
(original)
+++ poi/trunk/poi/src/main/java/org/apache/poi/hssf/usermodel/HSSFChart.java
Thu Dec 30 21:35:45 2021
@@ -218,7 +218,7 @@ public final class HSSFChart {
} else if(r instanceof SeriesTextRecord) {
// Applies to a series, unless we've seen a legend already
SeriesTextRecord str = (SeriesTextRecord)r;
- if(lastChart.legendRecord == null && lastChart.series.size() >
0) {
+ if(lastChart.legendRecord == null &&
!lastChart.series.isEmpty()) {
HSSFSeries series =
lastChart.series.get(lastChart.series.size()-1);
series.seriesTitleText = str;
} else {
Modified:
poi/trunk/poi/src/main/java/org/apache/poi/hssf/usermodel/HSSFSheet.java
URL:
http://svn.apache.org/viewvc/poi/trunk/poi/src/main/java/org/apache/poi/hssf/usermodel/HSSFSheet.java?rev=1896547&r1=1896546&r2=1896547&view=diff
==============================================================================
--- poi/trunk/poi/src/main/java/org/apache/poi/hssf/usermodel/HSSFSheet.java
(original)
+++ poi/trunk/poi/src/main/java/org/apache/poi/hssf/usermodel/HSSFSheet.java
Thu Dec 30 21:35:45 2021
@@ -300,7 +300,7 @@ public final class HSSFSheet implements
}
}
- if (_rows.size() > 0) {
+ if (!_rows.isEmpty()) {
Integer key = Integer.valueOf(row.getRowNum());
HSSFRow removedRow = _rows.remove(key);
if (removedRow != row) {
Modified:
poi/trunk/poi/src/main/java/org/apache/poi/poifs/property/DirectoryProperty.java
URL:
http://svn.apache.org/viewvc/poi/trunk/poi/src/main/java/org/apache/poi/poifs/property/DirectoryProperty.java?rev=1896547&r1=1896546&r2=1896547&view=diff
==============================================================================
---
poi/trunk/poi/src/main/java/org/apache/poi/poifs/property/DirectoryProperty.java
(original)
+++
poi/trunk/poi/src/main/java/org/apache/poi/poifs/property/DirectoryProperty.java
Thu Dec 30 21:35:45 2021
@@ -189,7 +189,7 @@ public class DirectoryProperty extends P
*/
protected void preWrite()
{
- if (_children.size() > 0)
+ if (!_children.isEmpty())
{
Property[] children = _children.toArray(new Property[ 0 ]);
Modified:
poi/trunk/poi/src/main/java/org/apache/poi/ss/format/CellNumberFormatter.java
URL:
http://svn.apache.org/viewvc/poi/trunk/poi/src/main/java/org/apache/poi/ss/format/CellNumberFormatter.java?rev=1896547&r1=1896546&r2=1896547&view=diff
==============================================================================
---
poi/trunk/poi/src/main/java/org/apache/poi/ss/format/CellNumberFormatter.java
(original)
+++
poi/trunk/poi/src/main/java/org/apache/poi/ss/format/CellNumberFormatter.java
Thu Dec 30 21:35:45 2021
@@ -261,7 +261,7 @@ public class CellNumberFormatter extends
first = false;
}
}
- if (fractionalSpecials.size() > 0) {
+ if (!fractionalSpecials.isEmpty()) {
fmtBuf.append('.');
for (Special s : fractionalSpecials) {
if (isDigitFmt(s)) {
@@ -822,7 +822,7 @@ public class CellNumberFormatter extends
private void writeFractional(StringBuffer result, StringBuffer output) {
int digit;
int strip;
- if (fractionalSpecials.size() > 0) {
+ if (!fractionalSpecials.isEmpty()) {
String decimalSeparator =
Character.toString(getDecimalFormatSymbols().getDecimalSeparator());
digit = result.indexOf(decimalSeparator) + 1;
if (exponent != null) {
Modified:
poi/trunk/poi/src/main/java/org/apache/poi/ss/format/CellNumberPartHandler.java
URL:
http://svn.apache.org/viewvc/poi/trunk/poi/src/main/java/org/apache/poi/ss/format/CellNumberPartHandler.java?rev=1896547&r1=1896546&r2=1896547&view=diff
==============================================================================
---
poi/trunk/poi/src/main/java/org/apache/poi/ss/format/CellNumberPartHandler.java
(original)
+++
poi/trunk/poi/src/main/java/org/apache/poi/ss/format/CellNumberPartHandler.java
Thu Dec 30 21:35:45 2021
@@ -49,7 +49,7 @@ public class CellNumberPartHandler imple
// See comment in writeScientific -- exponent handling is complex.
// (1) When parsing the format, remove the sign from after the 'e'
and
// put it before the first digit of the exponent.
- if (exponent == null && specials.size() > 0) {
+ if (exponent == null && !specials.isEmpty()) {
exponent = new Special('.', pos);
specials.add(exponent);
insertSignForExponent = part.charAt(1);
@@ -73,7 +73,7 @@ public class CellNumberPartHandler imple
break;
case '.':
- if (decimalPoint == null && specials.size() > 0) {
+ if (decimalPoint == null && !specials.isEmpty()) {
decimalPoint = new Special('.', pos);
specials.add(decimalPoint);
}
Modified:
poi/trunk/poi/src/test/java/org/apache/poi/poifs/filesystem/TestPOIFSFileSystem.java
URL:
http://svn.apache.org/viewvc/poi/trunk/poi/src/test/java/org/apache/poi/poifs/filesystem/TestPOIFSFileSystem.java?rev=1896547&r1=1896546&r2=1896547&view=diff
==============================================================================
---
poi/trunk/poi/src/test/java/org/apache/poi/poifs/filesystem/TestPOIFSFileSystem.java
(original)
+++
poi/trunk/poi/src/test/java/org/apache/poi/poifs/filesystem/TestPOIFSFileSystem.java
Thu Dec 30 21:35:45 2021
@@ -357,7 +357,7 @@ final class TestPOIFSFileSystem {
dis.reset();
}
}
- assertTrue(props.size() > 0);
+ assertFalse(props.isEmpty());
}
}
return count;
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
