sudeep87 / uimafit

Automatically exported from code.google.com/p/uimafit
0 stars 0 forks source link

Consider removing select*() using type names before 1.2.0 #66

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
With on of the pasts commits I have introduced select* methods in CasUtil that 
allow specifying a type by its name (as a String). Implementing all methods 
twice so that one accepts a type while the other accepts a type name causes the 
API to explode. However, since these methods are indeed convenient (even used 
in unit tests now), I would opt to remove them before 1.2.0 and again in favor 
of the query-like approach suggested in issue #65 (which will probably not be 
1.2.0 though).

Original issue reported on code.google.com by richard.eckart on 18 Mar 2011 at 3:51

GoogleCodeExporter commented 8 years ago
Replacing them with the issue 65 approach before 1.2.0 sounds good to me.

Original comment by steven.b...@gmail.com on 23 Mar 2011 at 10:48

GoogleCodeExporter commented 8 years ago
Removed them. Note that these methods were not available in 1.1.0, so that 
implementing issue 65 is not a prerequisite for removing them. For the same 
reason I will not record this issue in the CHANGES file.

Original comment by richard.eckart on 25 Mar 2011 at 10:55

GoogleCodeExporter commented 8 years ago
works for me.  thanks.

Original comment by phi...@ogren.info on 27 Mar 2011 at 2:59