You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@opennlp.apache.org by "Atita Arora (Jira)" <ji...@apache.org> on 2023/01/09 19:17:00 UTC

[jira] [Commented] (OPENNLP-1437) Change removeChar() in NumberUtil.java with String.replaceAll

    [ https://issues.apache.org/jira/browse/OPENNLP-1437?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17656253#comment-17656253 ] 

Atita Arora commented on OPENNLP-1437:
--------------------------------------

Shall I raise a PR replacing 
{code:java}
number = number.trim();
number = removeChar(number, ' ');{code}
with 
{code:java}
number = number.replaceAll(number, " ");  {code}
and delete the method removeChar().

> Change removeChar() in NumberUtil.java with String.replaceAll
> -------------------------------------------------------------
>
>                 Key: OPENNLP-1437
>                 URL: https://issues.apache.org/jira/browse/OPENNLP-1437
>             Project: OpenNLP
>          Issue Type: Task
>            Reporter: Jeff Zemerick
>            Priority: Major
>
> Change removeChar() in NumberUtil.java with String.replaceAll(). See the conversation on the pull request at [https://github.com/apache/opennlp/pull/479/files.]
>  



--
This message was sent by Atlassian Jira
(v8.20.10#820010)