Mailing-List: contact dev-help@poi.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "POI Developers List" <dev@poi.apache.org>
Subject: Re: [VOTE] Apache POI 3.15-beta3
To: POI Developers List <dev@poi.apache.org>
References: <57ABAEED.1010501@apache.org>
 <CABdJj55jQQnAp0bLJ=wMRS7E0fWzYXYig6RG+oP9Zr_LXnrrzw@mail.gmail.com>
 <712c7c0b-7279-2977-1bea-2bfeefeda834@corefiling.co.uk>
 <SN1PR09MB1069C8443A72A79FB1B61113C71F0@SN1PR09MB1069.namprd09.prod.outlook.com>
 <CAM+TppK=hj3=6+LG38gckcv87ADGN+19i9QV6KDLsDwMNQn_vg@mail.gmail.com>
 <CAM+TppLbXFbb9Sze4Sz9QS1Nn+z5jUSY8=hprRwiNH=0EnLV0w@mail.gmail.com>
 <CAM+Tpp+y0jHJ8mJgWG+987MbdeM=fOhdtHJV2i_5xDO=7q2+Aw@mail.gmail.com>
 <CAM+TppK-hBTpYtS8nXdOUZb_5+ezUb6rN2xSDgoLTrGfGzKRyQ@mail.gmail.com>
 <CAM+Tpp+32A-pVz=cSAbPPA1twEPsNfXsPhjky2Y_9upSUhqwHg@mail.gmail.com>
 <CABdJj55g4c9X6N3WxCcWrDHOBhXotSxNOFV4-4r27_GYLo9gcA@mail.gmail.com>
 <CAM+TppJ1E-EfGra21K3DVLnp4YKDzn4qgZkYh=-oOc2DdpSnBg@mail.gmail.com>
 <CAM+TppL--+8gvEAepZAzQBHvXD0xUKtxrBXRJZkbUbu1TKhCAw@mail.gmail.com>
 <CABdJj55-ObZ0PP+wXp0EB8FKkzM9MSX1n11dxqbqsS0x+_AJ=w@mail.gmail.com>
From: David North <dtn-poi@corefiling.co.uk>
Message-ID: <0272807e-ef6d-6b06-bd5b-9dc599ea44b4@corefiling.co.uk>
Date: Fri, 19 Aug 2016 09:30:23 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101
 Thunderbird/45.2.0
MIME-Version: 1.0
In-Reply-To: <CABdJj55-ObZ0PP+wXp0EB8FKkzM9MSX1n11dxqbqsS0x+_AJ=w@mail.gmail.com>
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
archived-at: Fri, 19 Aug 2016 08:30:36 -0000

Should be able to sort it this weekend, yes.

Thanks,
David

On 17/08/16 20:32, Dominik Stadler wrote:
> Hi,
> 
> As the anniversary of POI is getting near and it would be nice to have at
> least a 3.15 release for our 15th birthday, I suggest we build a 3.15-RC1
> from latest trunk so we include the two regressions that Javen fixed and
> the additional source-directory removed.
> 
> Then we might just be able to finish in time!
> 
> David, are you ok to run the release-steps a 2nd time?
> 
> Thanks... Dominik.
> 
> 
> On Tue, Aug 16, 2016 at 11:45 PM, Javen O'Neal <javenoneal@gmail.com> wrote:
> 
>> Fixed in r1756552.
>>
>> http://svn.apache.org/viewvc?view=revision&revision=1756552
>>
>> On Aug 15, 2016 9:09 AM, "Javen O'Neal" <javenoneal@gmail.com> wrote:
>>
>>>> * 4 times NullPointerException in XSLFTextParagraph.
>> getDefaultFontSize()
>>> I opened bug 60005 [1] to fix the NPE in XSLFTextParagraph.
>>> getDefaultFontSize()
>>> This has been fixed.
>>>
>>>> * 456 times: ArrayIndexOutOfBoundsException in
>> SprmOperation.getOperand()
>>> The assumption that the operand length is "surely shorter than an int"
>>> seems to incorrect for some files, since the test failed with AIOOB 4,
>>> meaning the operandLength is at least 5 bytes.
>>> I would need to look at the HWPF document specification ([MS-DOC].pdf,
>>> [3]) to see if the operandLength may be longer than an int.
>>> I do not know what the best way to fix this code is
>>>
>>>             // surely shorter than an int...
>>>             byte operandLength = _grpprl[_gOffset + 1];
>>>
>>>             // initialized to zeros by JVM
>>>             byte[] codeBytes = new byte[LittleEndian.INT_SIZE];
>>>             for ( int i = 0; i < operandLength; i++ )
>>>                 if ( _gOffset + i < _grpprl.length )
>>>                     codeBytes[i] = _grpprl[_gOffset + 1 + i];
>>>
>>>             return LittleEndian.getInt( codeBytes, 0 );
>>>
>>>
>>> [1] https://bz.apache.org/bugzilla/show_bug.cgi?id=60005
>>> [2] https://svn.apache.org/viewvc/poi/trunk/src/scratchpad/src/
>>> org/apache/poi/hwpf/sprm/SprmOperation.java?revision=
>>> 1753052&view=markup#l106
>>> [3] https://interoperability.blob.core.windows.net/files/
>>> OfficeFileFormatsProtocols.zip
>>>
>>> On Mon, Aug 15, 2016 at 3:09 AM, Dominik Stadler <dominik.stadler@gmx.at
>>>
>>> wrote:
>>>> Hi,
>>>>
>>>> Running the regression tests for POI 3.15-beta3 against the CommonCrawl
>>>> corpus is now finished, initial results are as follows:
>>>>
>>>> * 11966 fail because I did not add commons-collections4, I'll trigger a
>>>> re-run to get document-counts correctly show  the number of regressing
>>>> documents
>>>>
>>>> * 456 times: ArrayIndexOutOfBoundsException in
>> SprmOperation.getOperand()
>>>>
>>>> java.lang.RuntimeException: java.lang.ArrayIndexOutOfBoundsException:
>> *
>>>>         at o.a.p.hwpf.extractor.WordExtractor.getText(
>>> WordExtractor.java:317)
>>>>         at o.a.p.stress.AbstractFileHandler.handleExtractingInternal(
>>> AbstractFileHandler.java:85)
>>>>         at o.a.p.stress.AbstractFileHandler.handleExtracting(
>>> AbstractFileHandler.java:60)
>>>>         at org.dstadler.commoncrawl.FileHandlingRunnable.run(
>>> FileHandlingRunnable.java:58)
>>>>
>>>> Caused by: java.lang.ArrayIndexOutOfBoundsException: 4
>>>>         at o.a.p.hwpf.sprm.SprmOperation.
>> getOperand(SprmOperation.java:
>>> 113)
>>>>         at o.a.p.hwpf.sprm.SectionSprmUncompressor.
>>> unCompressSEPOperation(SectionSprmUncompressor.java:62)
>>>>         at o.a.p.hwpf.sprm.SectionSprmUncompressor.uncompressSEP(
>>> SectionSprmUncompressor.java:44)
>>>>         at o.a.p.hwpf.model.SEPX.getSectionProperties(SEPX.java:61)
>>>>         at o.a.p.hwpf.usermodel.Section.(Section.java:36)
>>>>         at o.a.p.hwpf.usermodel.Range.getSection(Range.java:745)
>>>>         at o.a.p.hwpf.converter.AbstractWordConverter.processDocument(
>>> AbstractWordConverter.java:721)
>>>>         at o.a.p.hwpf.extractor.WordExtractor.getText(
>>> WordExtractor.java:299)
>>>>         ... 9 more
>>>>
>>>> * 4 times NullPointerException in XSLFTextParagraph.
>> getDefaultFontSize()
>>>>
>>>> java.lang.NullPointerException
>>>>         at o.a.p.xslf.usermodel.XSLFTextParagraph.getDefaultFontSize(
>>> XSLFTextParagraph.java:935)
>>>>         at o.a.p.sl.draw.DrawTextParagraph.getAttributedString(
>>> DrawTextParagraph.java:567)
>>>>         at o.a.p.sl.draw.DrawTextParagraph.breakText(
>>> DrawTextParagraph.java:235)
>>>>         at o.a.p.sl.draw.DrawTextShape.drawParagraphs(DrawTextShape.
>>> java:158)
>>>>         at o.a.p.sl.draw.DrawTextShape.getTextHeight(DrawTextShape.
>>> java:219)
>>>>         at o.a.p.sl.draw.DrawTextShape.drawContent(DrawTextShape.
>>> java:102)
>>>>         at o.a.p.sl.draw.DrawSimpleShape.draw(DrawSimpleShape.java:93)
>>>>         at o.a.p.sl.draw.DrawSheet.draw(DrawSheet.java:67)
>>>>         at o.a.p.sl.draw.DrawSlide.draw(DrawSlide.java:39)
>>>>         at o.a.p.xslf.usermodel.XSLFSlide.draw(XSLFSlide.java:301)
>>>>         at o.a.p.stress.SlideShowHandler.
>> renderSlides(SlideShowHandler.
>>> java:120)
>>>>         at o.a.p.stress.SlideShowHandler.handleSlideShow(
>>> SlideShowHandler.java:43)
>>>>         at o.a.p.stress.XSLFFileHandler.handleFile(XSLFFileHandler.
>>> java:43)
>>>>         at org.dstadler.commoncrawl.FileHandlingRunnable.run(
>>> FileHandlingRunnable.java:58)
>>>>
>>>>
>>>>
>>>> The others are probably flaky things where files caused OOM/Timeout
>>> before
>>>> and thus were not reported with these errors before.
>>>>
>>>>
>>>> See http://people.apache.org/~centic/poi_regression/reports/ and
>>>> http://people.apache.org/~centic/poi_regression/reportsAll/ for
>> detailed
>>>> results.
>>>
>>
> 

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org