Bug 58220

Summary: XSSFFont getFontHeight javadoc says "in points" but should say "in EMUs"
Product: POI Reporter: petteyg359
Component: XSSFAssignee: POI Developers List <dev>
Status: RESOLVED FIXED    
Severity: trivial    
Priority: P2    
Version: unspecified   
Target Milestone: ---   
Hardware: All   
OS: All   

Description petteyg359 2015-08-06 19:44:40 UTC
The Javadoc should specify the right unit that the method actually returns.
Comment 1 Nick Burch 2015-08-12 13:05:56 UTC
I don't think they are EMUs though, are they?

From a quick check, the return looks to be in 20th points, just as with HSSF, and as described at https://poi.apache.org/apidocs/org/apache/poi/ss/usermodel/Font.html#getFontHeight%28%29

I'd propose replacing the XSSFFont javadoc with the Font or HSSFFont one, would that solve the issue for you?
Comment 2 petteyg359 2015-08-12 13:59:13 UTC
That's probably best. Looking at the source, getFontHeightInPoints has nothing but "@see #getFontHeight" which is rather inadequate documentation. HSSFFont documentation actually exists :)
Comment 3 Nick Burch 2015-08-12 14:14:54 UTC
Fixed in r1695528, and site republished with the updates in, thanks for your advice!