[OPENNLP-1180] Use String[] instead of StringList in LanguageModel API

• Type: Task
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.9.0
• Component/s: language model
• Assignee: Tommaso Teofili
• Reporter: Tommaso Teofili
• Created: 08/Jan/18 20:16
• Updated: 11/May/18 04:18
• Resolved: 09/Jan/18 17:32

Description

Current LanguageModel API uses StringList, however that's less convenient for easy consumption as one needs to look into StringList and adapt its code to convert arrays or collections of Strings into StringList. Additionally this requires more objects to be created that will be soon discarded by garbage collection e.g. the input StringList for LM#calculateProbability and LM#predictNextTokens.
I propose to deprecate those methods and add new ones with exactly the same signature but using String[] (or String...) instead.

Internally StringLists can be kept or not, but that would be an implementation detail and allows to move away from using them more easily.

Issue Links

• links to: GitHub Pull Request #304