Bug 61062

Summary: Various HPSF related fixes
Product: POI Reporter: Andreas Beeker <kiwiwings>
Component: HPSFAssignee: POI Developers List <dev>
Status: RESOLVED FIXED    
Severity: normal    
Priority: P2    
Version: 3.17-dev   
Target Milestone: ---   
Hardware: All   
OS: All   

Description Andreas Beeker 2017-05-02 23:18:09 UTC
This is a container entry for various HPSF related fixes.
The patches are documented below...
Comment 1 Andreas Beeker 2017-05-02 23:46:17 UTC
Use LittleEndianStream instead of offset calculation:
the old implementation needed to update the offset value with every written fragment - using a the LittleEndianStream wraps that offset calculation.
Furthermore the 0-padding was ignored on some places, this has been fixed here too
r1793594

IntegrationTest - move excludes to file handler:
the integration tests had all excludes in the TestAllFiles.
now the filename is also passed down to the handlers which enables the excluding inside the handlers
r1793595

Fix hpsf javadocs and code issues:
the javadocs don't need to be html compliant - see http://www.oracle.com/technetwork/articles/java/index-137868.html
Furthermore there are some sonarcube code fixes and a mapper for locale ids
r1793596

Enable new number types in VariantSupport:
Added a few more Variant mappings for number types
r1793598

Add CopyCompare to IntegrationTests
r1793600
Comment 2 Andreas Beeker 2017-05-03 19:21:56 UTC
HPSF: Change CustomProperties to delegate - r1793699
the old implementation (by extending HashMap) left a few open interpretations open, i.e when is a CustomProperty vs. an property value returned in the various Map methods.
I hope this is more clear now.