Bug 61684

Summary: CellStyle exposes colors as short and Color instances
Product: POI Reporter: PJ Fanning <fanningpj>
Component: SL CommonAssignee: POI Developers List <dev>
Status: NEW ---    
Severity: enhancement    
Priority: P2    
Version: 3.17-dev   
Target Milestone: ---   
Hardware: PC   
OS: Mac OS X 10.1   

Description PJ Fanning 2017-10-27 18:54:27 UTC
short getFillBackgroundColor();
    Color getFillBackgroundColorColor();

Do we want to continue to support both?
With BorderStyle and FillPatternType, we no longer support short values.
Comment 1 Javen O'Neal 2017-10-27 19:10:20 UTC
Does short refer to an index into one of the tables defined in styles.xml? Can short be arbitrarily large (within file format limits) if there are a large number of styles, or is it a fixed quantity (only works for predefined quantities), which could be captured by a color enum or static class constant)?

If so, this would be different BorderStyle and FillPatternType enums where the number is irrelevant (either a POI implementation detail or a format implementation detail).