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

ASF GitHub Bot commented on OPENNLP-1410:
-----------------------------------------

jzonthemtn commented on code in PR #456:
URL: https://github.com/apache/opennlp/pull/456#discussion_r1045808696


##########
opennlp-tools/src/main/java/opennlp/tools/namefind/TokenNameFinderFactory.java:
##########
@@ -118,9 +160,7 @@ public static TokenNameFinderFactory create(String 
subclassName, byte[] featureG
             TokenNameFinderFactory.class, subclassName);
       } catch (Exception e) {
         String msg = "Could not instantiate the " + subclassName
-            + ". The initialization throw an exception.";
-        System.err.println(msg);
-        e.printStackTrace();
+            + ". The initialization threw an exception.";
         throw new InvalidFormatException(msg, e);

Review Comment:
   I see - it can still throw `InvalidFormatException` here.





> Enhance JavaDoc in opennlp.tools.namefind package
> -------------------------------------------------
>
>                 Key: OPENNLP-1410
>                 URL: https://issues.apache.org/jira/browse/OPENNLP-1410
>             Project: OpenNLP
>          Issue Type: Improvement
>          Components: Name Finder
>    Affects Versions: 2.1.0
>            Reporter: Martin Wiesner
>            Assignee: Martin Wiesner
>            Priority: Minor
>             Fix For: 2.1.1
>
>
> The JavaDoc the _opennlp.tools.namefind_ package suffers from several 
> inconsistencies and missing descriptions. Moreover, several typos are present 
> that need sanitizing.
> It needs enhancements and/or additions to provide more clarity for readers.



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

Reply via email to