[
https://issues.apache.org/jira/browse/OPENNLP-1434?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17653681#comment-17653681
]
ASF GitHub Bot commented on OPENNLP-1434:
-----------------------------------------
rzo1 commented on code in PR #479:
URL: https://github.com/apache/opennlp/pull/479#discussion_r1060162143
##########
opennlp-uima/src/main/java/opennlp/uima/normalizer/NumberUtil.java:
##########
@@ -52,31 +51,36 @@ public static boolean isLanguageSupported(String
languageCode) {
}
/**
- * Removes trailing and containing space.
+ * Removes a character from given string {@code s}.
+ *
+ * @param s The string to process.
+ * @param remove The {@link Character} to remove from {@code s}.
*/
- private static String removeChar(String string, char remove) {
+ private static String removeChar(String s, char remove) {
Review Comment:
This is basically a `s.replace(remove, "")` (replace all occurences of the
given `char` in the given `String` literal). Sadly, were is no
`String#replace(char,String)` available, but we could do
`s.replace(Character.toString(remove), "")` (or change the method signature).
No blocker, but would reduce code complexity.
> Enhance JavaDoc in opennlp.uima packages
> ----------------------------------------
>
> Key: OPENNLP-1434
> URL: https://issues.apache.org/jira/browse/OPENNLP-1434
> Project: OpenNLP
> Issue Type: Improvement
> Components: UIMA Integration
> Affects Versions: 2.1.0
> Reporter: Martin Wiesner
> Assignee: Martin Wiesner
> Priority: Minor
> Fix For: 2.1.1
>
>
> The JavaDoc the _opennlp.uima.*_ packages suffer from several inconsistencies
> and missing descriptions. Moreover, several typos are present that need
> sanitizing.
> The classes, interfaces, etc., require enhancements and/or additions to
> provide more clarity for readers.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
