moparisthebest/poi
1
0
Fork 0
Code Issues Pull Requests Releases 1 Wiki Activity

Bug 56195: replace System.err with propert logging

Browse Source 
git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1578147 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
Dominik Stadler 2014-03-16 19:46:56 +00:00
parent 78665403c5
commit ee58bd0d32
1 changed files with 7 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/ooxml/java/org/apache/poi/xssf/eventusermodel/XSSFSheetXMLHandler.java
View File

@ -18,6 +18,8 @@ package org.apache.poi.xssf.eventusermodel;
import org.apache.poi.ss.usermodel.BuiltinFormats; import org.apache.poi.ss.usermodel.BuiltinFormats;
import org.apache.poi.ss.usermodel.DataFormatter; import org.apache.poi.ss.usermodel.DataFormatter;
import org.apache.poi.util.POILogFactory;
import org.apache.poi.util.POILogger;
import org.apache.poi.xssf.model.StylesTable; import org.apache.poi.xssf.model.StylesTable;
import org.apache.poi.xssf.usermodel.XSSFCellStyle; import org.apache.poi.xssf.usermodel.XSSFCellStyle;
import org.apache.poi.xssf.usermodel.XSSFRichTextString; import org.apache.poi.xssf.usermodel.XSSFRichTextString;
@ -31,7 +33,9 @@ import org.xml.sax.helpers.DefaultHandler;
* row and cell events for it. * row and cell events for it.
*/ */
public class XSSFSheetXMLHandler extends DefaultHandler { public class XSSFSheetXMLHandler extends DefaultHandler {
/** private static final POILogger logger = POILogFactory.getLogger(XSSFSheetXMLHandler.class);
/**
* These are the different kinds of cells we support. * These are the different kinds of cells we support.
* We keep track of the current one between * We keep track of the current one between
* the start and end. * the start and end.
@ -168,7 +172,7 @@ public class XSSFSheetXMLHandler extends DefaultHandler {
// TODO Retrieve the shared formula and tweak it to // TODO Retrieve the shared formula and tweak it to
// match the current cell // match the current cell
if(formulasNotResults) { if(formulasNotResults) {
System.err.println("Warning - shared formulas not yet supported!"); logger.log(POILogger.WARN, "shared formulas not yet supported!");
} else { } else {
// It's a shared formula, so we can't get at the formula string yet // It's a shared formula, so we can't get at the formula string yet
// However, they don't care about the formula string, so that's ok! // However, they don't care about the formula string, so that's ok!
@ -283,7 +287,7 @@ public class XSSFSheetXMLHandler extends DefaultHandler {
thisStr = rtss.toString(); thisStr = rtss.toString();
} }
catch (NumberFormatException ex) { catch (NumberFormatException ex) {
System.err.println("Failed to parse SST index '" + sstIndex + "': " + ex.toString()); logger.log(POILogger.ERROR, "Failed to parse SST index '" + sstIndex, ex);
} }
break; break;