search

fangfangli / cleartk

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

Remove type parameter from ContextExtractor #291

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
ContextExtractor doesn't need to have a type parameter anymore. We should 
remove it, but that will break backwards compatibility. So we should only do 
this in a release where we're making other backwards-incompatible changes.

Original issue reported on code.google.com by steven.b...@gmail.com on 12 Feb 2012 at 11:55

GoogleCodeExporter commented 9 years ago 
One way to do this without breaking backwards compatibility would be:

* Create a copy of ContextExtractor with no type parameter and a new name

* Deprecate the original ContextExtractor and tell everyone to use the new one

I think this is probably what we should do, but I don't know what else to call 
it...

Original comment by steven.b...@gmail.com on 22 Apr 2012 at 9:13

GoogleCodeExporter commented 9 years ago

Original comment by steven.b...@gmail.com on 24 Jul 2012 at 5:36

GoogleCodeExporter commented 9 years ago 
Another solution would be to just change the package name. We could copy it to 
the "simple" sub-package since it does implement SimpleFeatureExtractor.

Original comment by steven.b...@gmail.com on 26 Jul 2012 at 5:23

GoogleCodeExporter commented 9 years ago 
We've agreed to rename it to CleartkExtractor.

Original comment by steven.b...@gmail.com on 4 Aug 2012 at 4:27

GoogleCodeExporter commented 9 years ago 
This issue was closed by revision r3961.

Original comment by steven.b...@gmail.com on 4 Aug 2012 at 6:04