You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@poi.apache.org by bu...@apache.org on 2016/07/10 06:51:05 UTC
[Bug 59836] New: Replace primitives with enums
https://bz.apache.org/bugzilla/show_bug.cgi?id=59836
Bug ID: 59836
Summary: Replace primitives with enums
Product: POI
Version: 3.15-dev
Hardware: PC
OS: Linux
Status: NEW
Severity: enhancement
Priority: P2
Component: SS Common
Assignee: dev@poi.apache.org
Reporter: onealj@apache.org
This is a aggregate bug for recent changes in classes from using short or int
to using enums with codes.
Benefits:
* self-explanatory usage rather than needing to list the valid constants in the
javadocs
* easier to distinguish arguments in function signatures (not just a 5 ints)
* clearer error messages, no functions needed to convert code to string
* free range checking (less boiler plate code that does the same)
* type safety
Drawbacks:
* some permgen overhead
* type checking execution overhead
Overall, this seems like a good thing.
In effort to maintain backwards compatibility, the following is suggested:
If there is currently
int getSomething();
void setSomething(int);
Then we should add
enum getSomethingEnum();
void setSomething(Enum);
And deprecate
void setSomething(int);
int getSomething();
It's a bit tricky handling getSomething/getSomethingEnum while maintaining
backwards compatibility because at some point the signature will abruptly
change. Both getters should be offered for a while with a note encouraging
users to use the constants or enums rather than using literal values. This is
good coding practice anyway to not hard-code literal values, but is especially
important with regard to backwards compatibility here.
At least 2 releases later, but not later than POI 4.0:
Delete void setSomething(int);
Delete int getSomething();
Rename enum getSomethingEnum(); to enum getSomething();
In the end, we will have
enum getSomething();
void setSomething(enum);
as the only two methods.
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org
[Bug 59836] Tracker: Replace primitives with enums
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59836
Javen O'Neal <on...@apache.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Depends on| |59873
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org
[Bug 59836] Tracker: Replace primitives with enums
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59836
Bug 59836 depends on bug 59791, which changed state.
Bug 59791 Summary: Convert Cell Type to an enum
https://bz.apache.org/bugzilla/show_bug.cgi?id=59791
What |Removed |Added
----------------------------------------------------------------------------
Status|NEEDINFO |RESOLVED
Resolution|--- |FIXED
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org
Re: [Bug 59836] New: Replace primitives with enums
Posted by Javen O'Neal <ja...@gmail.com>.
Unfortunately, Java language doesn't allow enums to be subclassed or
created via a factory.
Enums need to be statically defined at run time so that equality checks and
switch statements can be resolved at compile time rather than run time.
This is a Java 6 requirement.
On Jul 10, 2016 7:29 AM, "Mark Murphy" <jm...@gmail.com> wrote:
> Maybe we should have a standard enum template that defines everything that
> belongs in the enum class. It would be easiest if enums could inherit from
> each other, or if they weren't final classes, but that doesn't work. I am
> thinking of some conversion from and to the simple type underlaying it, but
> that doesn't work for the binary formats. Maybe the better way is to have a
> conversion class that takes the various enums and converts them from/to the
> various simple types.
>
> On Sun, Jul 10, 2016 at 2:51 AM, <bu...@apache.org> wrote:
>
> > https://bz.apache.org/bugzilla/show_bug.cgi?id=59836
> >
> > Bug ID: 59836
> > Summary: Replace primitives with enums
> > Product: POI
> > Version: 3.15-dev
> > Hardware: PC
> > OS: Linux
> > Status: NEW
> > Severity: enhancement
> > Priority: P2
> > Component: SS Common
> > Assignee: dev@poi.apache.org
> > Reporter: onealj@apache.org
> >
> > This is a aggregate bug for recent changes in classes from using short or
> > int
> > to using enums with codes.
> >
> > Benefits:
> > * self-explanatory usage rather than needing to list the valid constants
> > in the
> > javadocs
> > * easier to distinguish arguments in function signatures (not just a 5
> > ints)
> > * clearer error messages, no functions needed to convert code to string
> > * free range checking (less boiler plate code that does the same)
> > * type safety
> >
> > Drawbacks:
> > * some permgen overhead
> > * type checking execution overhead
> >
> > Overall, this seems like a good thing.
> >
> > In effort to maintain backwards compatibility, the following is
> suggested:
> >
> > If there is currently
> > int getSomething();
> > void setSomething(int);
> >
> > Then we should add
> > enum getSomethingEnum();
> > void setSomething(Enum);
> >
> > And deprecate
> > void setSomething(int);
> > int getSomething();
> >
> > It's a bit tricky handling getSomething/getSomethingEnum while
> maintaining
> > backwards compatibility because at some point the signature will abruptly
> > change. Both getters should be offered for a while with a note
> encouraging
> > users to use the constants or enums rather than using literal values.
> This
> > is
> > good coding practice anyway to not hard-code literal values, but is
> > especially
> > important with regard to backwards compatibility here.
> >
> > At least 2 releases later, but not later than POI 4.0:
> > Delete void setSomething(int);
> > Delete int getSomething();
> > Rename enum getSomethingEnum(); to enum getSomething();
> >
> > In the end, we will have
> > enum getSomething();
> > void setSomething(enum);
> > as the only two methods.
> >
> > --
> > You are receiving this mail because:
> > You are the assignee for the bug.
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
> > For additional commands, e-mail: dev-help@poi.apache.org
> >
> >
>
Re: [Bug 59836] New: Replace primitives with enums
Posted by Mark Murphy <jm...@gmail.com>.
Maybe we should have a standard enum template that defines everything that
belongs in the enum class. It would be easiest if enums could inherit from
each other, or if they weren't final classes, but that doesn't work. I am
thinking of some conversion from and to the simple type underlaying it, but
that doesn't work for the binary formats. Maybe the better way is to have a
conversion class that takes the various enums and converts them from/to the
various simple types.
On Sun, Jul 10, 2016 at 2:51 AM, <bu...@apache.org> wrote:
> https://bz.apache.org/bugzilla/show_bug.cgi?id=59836
>
> Bug ID: 59836
> Summary: Replace primitives with enums
> Product: POI
> Version: 3.15-dev
> Hardware: PC
> OS: Linux
> Status: NEW
> Severity: enhancement
> Priority: P2
> Component: SS Common
> Assignee: dev@poi.apache.org
> Reporter: onealj@apache.org
>
> This is a aggregate bug for recent changes in classes from using short or
> int
> to using enums with codes.
>
> Benefits:
> * self-explanatory usage rather than needing to list the valid constants
> in the
> javadocs
> * easier to distinguish arguments in function signatures (not just a 5
> ints)
> * clearer error messages, no functions needed to convert code to string
> * free range checking (less boiler plate code that does the same)
> * type safety
>
> Drawbacks:
> * some permgen overhead
> * type checking execution overhead
>
> Overall, this seems like a good thing.
>
> In effort to maintain backwards compatibility, the following is suggested:
>
> If there is currently
> int getSomething();
> void setSomething(int);
>
> Then we should add
> enum getSomethingEnum();
> void setSomething(Enum);
>
> And deprecate
> void setSomething(int);
> int getSomething();
>
> It's a bit tricky handling getSomething/getSomethingEnum while maintaining
> backwards compatibility because at some point the signature will abruptly
> change. Both getters should be offered for a while with a note encouraging
> users to use the constants or enums rather than using literal values. This
> is
> good coding practice anyway to not hard-code literal values, but is
> especially
> important with regard to backwards compatibility here.
>
> At least 2 releases later, but not later than POI 4.0:
> Delete void setSomething(int);
> Delete int getSomething();
> Rename enum getSomethingEnum(); to enum getSomething();
>
> In the end, we will have
> enum getSomething();
> void setSomething(enum);
> as the only two methods.
>
> --
> You are receiving this mail because:
> You are the assignee for the bug.
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
> For additional commands, e-mail: dev-help@poi.apache.org
>
>
[Bug 59836] Tracker: Replace primitives with enums
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59836
Bug 59836 depends on bug 61940, which changed state.
Bug 61940 Summary: Replace ClassID statics with enum
https://bz.apache.org/bugzilla/show_bug.cgi?id=61940
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution|--- |FIXED
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org
[Bug 59836] Tracker: Replace primitives with enums
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59836
Javen O'Neal <on...@apache.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Depends on| |60795
Referenced Bugs:
https://bz.apache.org/bugzilla/show_bug.cgi?id=60795
[Bug 60795] Add enum for msg types
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org
[Bug 59836] Tracker: Replace primitives with enums
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59836
Bug 59836 depends on bug 60187, which changed state.
Bug 60187 Summary: RegionUtil should use new enum style classes
https://bz.apache.org/bugzilla/show_bug.cgi?id=60187
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution|--- |FIXED
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org
[Bug 59836] Tracker: Replace primitives with enums
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59836
Bug 59836 depends on bug 59873, which changed state.
Bug 59873 Summary: Replace Hyperlink type int constants with a HyperlinkType enum
https://bz.apache.org/bugzilla/show_bug.cgi?id=59873
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution|--- |FIXED
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org
[Bug 59836] Tracker: Replace primitives with enums
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59836
Bug 59836 depends on bug 60228, which changed state.
Bug 60228 Summary: Cell.getCellTypeEnum should not be deprecated until Cell.getCellType returns a CellType
https://bz.apache.org/bugzilla/show_bug.cgi?id=60228
What |Removed |Added
----------------------------------------------------------------------------
Status|RESOLVED |UNCONFIRMED
Resolution|FIXED |---
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org
[Bug 59836] Tracker: Replace primitives with enums
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59836
--- Comment #3 from Andreas Beeker <ki...@apache.org> ---
preparation for removing those many HSSFColor subclasses with an enum via
r1779866
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org
[Bug 59836] Tracker: Replace primitives with enums
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59836
Javen O'Neal <on...@apache.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Depends on| |55330
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org
[Bug 59836] Tracker: Replace primitives with enums
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59836
Bug 59836 depends on bug 60803, which changed state.
Bug 60803 Summary: in XSSF files getErrorStyle() and setErrorStyle() have mismatching enum values
https://bz.apache.org/bugzilla/show_bug.cgi?id=60803
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution|--- |DUPLICATE
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org
[Bug 59836] Tracker: Replace primitives with enums
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59836
Javen O'Neal <on...@apache.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Depends on| |60187
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org
[Bug 59836] Tracker: Replace primitives with enums
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59836
--- Comment #2 from Javen O'Neal <on...@apache.org> ---
https://poi.apache.org/apidocs/org/apache/poi/xwpf/usermodel/XWPFPictureData.html#getPictureType()
XWPFPictureData.getPictureType() could return
https://poi.apache.org/apidocs/org/apache/poi/sl/usermodel/PictureData.PictureType.html
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org
[Bug 59836] Tracker: Replace primitives with enums
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59836
Javen O'Neal <on...@apache.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Depends on| |59877
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org
[Bug 59836] Tracker: Replace primitives with enums
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59836
Bug 59836 depends on bug 61898, which changed state.
Bug 61898 Summary: Quick Guide is getting a bit out of date
https://bz.apache.org/bugzilla/show_bug.cgi?id=61898
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution|--- |FIXED
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org
[Bug 59836] Tracker: Replace primitives with enums
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59836
Javen O'Neal <on...@apache.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Depends on| |59907
Referenced Bugs:
https://bz.apache.org/bugzilla/show_bug.cgi?id=59907
[Bug 59907] HSSFClientAnchor.setAnchorType API was broken (problem with
jasperreports-5.1.0 and poi-3.14)
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org
[Bug 59836] Tracker: Replace primitives with enums
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59836
Bug 59836 depends on bug 60228, which changed state.
Bug 60228 Summary: Cell.getCellTypeEnum should not be deprecated until Cell.getCellType returns a CellType
https://bz.apache.org/bugzilla/show_bug.cgi?id=60228
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution|--- |FIXED
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org
[Bug 59836] Tracker: Replace primitives with enums
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59836
Andreas Beeker <ki...@apache.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Depends on| |61940
Referenced Bugs:
https://bz.apache.org/bugzilla/show_bug.cgi?id=61940
[Bug 61940] Replace ClassID statics with enum
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org
[Bug 59836] Tracker: Replace primitives with enums
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59836
Bug 59836 depends on bug 60795, which changed state.
Bug 60795 Summary: Add enum for msg types
https://bz.apache.org/bugzilla/show_bug.cgi?id=60795
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution|--- |FIXED
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org
[Bug 59836] Tracker: Replace primitives with enums
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59836
Bug 59836 depends on bug 60228, which changed state.
Bug 60228 Summary: Cell.getCellTypeEnum should not be deprecated until Cell.getCellType returns a CellType
https://bz.apache.org/bugzilla/show_bug.cgi?id=60228
What |Removed |Added
----------------------------------------------------------------------------
Status|UNCONFIRMED |RESOLVED
Resolution|--- |FIXED
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org
[Bug 59836] Tracker: Replace primitives with enums
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59836
Javen O'Neal <on...@apache.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Component|SS Common |POI Overall
Depends on| |59264, 58671, 59833, 59790,
| |58636, 58190, 59791
Summary|Replace primitives with |Tracker: Replace primitives
|enums |with enums
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org
[Bug 59836] Tracker: Replace primitives with enums
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59836
Javen O'Neal <on...@apache.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Depends on| |59837
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org
[Bug 59836] Tracker: Replace primitives with enums
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59836
--- Comment #1 from Javen O'Neal <on...@apache.org> ---
Todo: update https://poi.apache.org/spreadsheet/quick-guide.html
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org
[Bug 59836] Tracker: Replace primitives with enums
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59836
Javen O'Neal <on...@apache.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Depends on| |60605
Referenced Bugs:
https://bz.apache.org/bugzilla/show_bug.cgi?id=60605
[Bug 60605] Convert Workbook.SHEET_STATE_HIDDEN, VISIBLE, and VERY_HIDDEN to
enum
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org
[Bug 59836] Tracker: Replace primitives with enums
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59836
Bug 59836 depends on bug 55330, which changed state.
Bug 55330 Summary: [PATCH] add PageMargin enum
https://bz.apache.org/bugzilla/show_bug.cgi?id=55330
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution|--- |FIXED
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org
[Bug 59836] Tracker: Replace primitives with enums
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59836
Javen O'Neal <on...@apache.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Depends on| |60136
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org
[Bug 59836] Tracker: Replace primitives with enums
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59836
Bug 59836 depends on bug 58190, which changed state.
Bug 58190 Summary: [PATCH] The current picture handling uses raw integers for types and index, replace with enum and reference
https://bz.apache.org/bugzilla/show_bug.cgi?id=58190
What |Removed |Added
----------------------------------------------------------------------------
Status|NEEDINFO |RESOLVED
Resolution|--- |FIXED
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org
[Bug 59836] Tracker: Replace primitives with enums
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59836
Javen O'Neal <on...@apache.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Depends on| |60803
Referenced Bugs:
https://bz.apache.org/bugzilla/show_bug.cgi?id=60803
[Bug 60803] in XSSF files getErrorStyle() and setErrorStyle() have mismatching
enum values
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org
[Bug 59836] Tracker: Replace primitives with enums
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59836
Javen O'Neal <on...@apache.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Depends on| |60228
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org
[Bug 59836] Tracker: Replace primitives with enums
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59836
Bug 59836 depends on bug 59837, which changed state.
Bug 59837 Summary: Replace CellStyle alignment constants with HorizontalAlignment and VerticalAlignment enums
https://bz.apache.org/bugzilla/show_bug.cgi?id=59837
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution|--- |FIXED
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org
[Bug 59836] Tracker: Replace primitives with enums
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59836
Greg Woolsey <gr...@gmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Depends on| |60666
Referenced Bugs:
https://bz.apache.org/bugzilla/show_bug.cgi?id=60666
[Bug 60666] DataValidationConstraint contains two type classes that are not
enums
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org
[Bug 59836] Tracker: Replace primitives with enums
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59836
Javen O'Neal <on...@apache.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Depends on| |61898
Referenced Bugs:
https://bz.apache.org/bugzilla/show_bug.cgi?id=61898
[Bug 61898] Quick Guide is getting a bit out of date
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org