Bug 61684 - CellStyle exposes colors as short and Color instances
Summary: CellStyle exposes colors as short and Color instances
Status: NEW
Alias: None
Product: POI
Classification: Unclassified
Component: SL Common (show other bugs)
Version: 3.17-dev
Hardware: PC Mac OS X 10.1
: P2 enhancement (vote)
Target Milestone: ---
Assignee: POI Developers List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-10-27 18:54 UTC by PJ Fanning
Modified: 2017-11-04 09:15 UTC (History)
0 users


Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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).