]> source.dussan.org Git - poi.git/commitdiff
Reverted DocumentSummary hack in codepage guessing code
authorAndreas Beeker <kiwiwings@apache.org>
Tue, 2 May 2017 23:30:17 +0000 (23:30 +0000)
committerAndreas Beeker <kiwiwings@apache.org>
Tue, 2 May 2017 23:30:17 +0000 (23:30 +0000)
git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1793601 13f79535-47bb-0310-9956-ffa450edef68

src/scratchpad/src/org/apache/poi/hwpf/HWPFOldDocument.java

index cfc010206280e83e6e664cb1277c4c258198e713..063d8b7b0a137e32873b7656bcee1cbb53bda9bb 100644 (file)
@@ -198,36 +198,7 @@ public class HWPFOldDocument extends HWPFDocumentCore {
      * @return The detected Charset from the old font table
      */
     private Charset guessCodePage(OldFontTable fontTable) {
-        //try to get it out of the overall document summary information
-        DocumentSummaryInformation summaryInformation = getDocumentSummaryInformation();
-        if (summaryInformation != null) {
-            CustomProperties customProperties = summaryInformation.getCustomProperties();
-            if (customProperties != null) {
-                int codePage = customProperties.getCodepage();
-                if (codePage > -1) {
-                    try {
-                        return Charset.forName(CodePageUtil.codepageToEncoding(codePage));
-                    } catch (UnsupportedEncodingException e) {
-                        //swallow
-                    }
-                }
-            }
-            //If that didn't work, for now, try to get first valid code page in a valid section
-            for (Section section : summaryInformation.getSections()) {
-                if (section.getOffset() < 0) {
-                    continue;
-                }
-                int codePage = section.getCodepage();
-                if (codePage > -1) {
-                    try {
-                        return Charset.forName(CodePageUtil.codepageToEncoding(codePage));
-                    } catch (UnsupportedEncodingException e) {
-                        //swallow
-                    }
-                }
-            }
-        }
-        //if that still doesn't work, pick the first non-default, non-symbol charset
+        // pick the first non-default, non-symbol charset
         for (OldFfn oldFfn : fontTable.getFontNames()) {
             HwmfFont.WmfCharset wmfCharset = HwmfFont.WmfCharset.valueOf(oldFfn.getChs()& 0xff);
             if (wmfCharset != null &&