[
https://issues.apache.org/jira/browse/OPENNLP-1413?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17648584#comment-17648584
]
ASF GitHub Bot commented on OPENNLP-1413:
-----------------------------------------
rzo1 commented on code in PR #459:
URL: https://github.com/apache/opennlp/pull/459#discussion_r1050621640
##########
opennlp-tools/src/main/java/opennlp/tools/util/ext/ExtensionLoader.java:
##########
@@ -41,20 +42,23 @@ static void setOSGiAvailable() {
// Pass in the type (interface) of the class to load
/**
- * Instantiates an user provided extension to OpenNLP.
+ * Instantiates a user provided extension to OpenNLP.
* <p>
* The extension is either loaded from the class path or if running
* inside an OSGi environment via an OSGi service.
* <p>
- * Initially it tries using the public default
- * constructor. If it is not found, it will check if the class follows the
singleton
- * pattern: a static field named <code>INSTANCE</code> that returns an
object of the type
- * <code>T</code>.
+ * Initially, the load is conducted using the public no-arg constructor.
+ * If no such constructor is not found, it is checked if the class follows
the
+ * {@code Singleton} pattern: a static field named {@code INSTANCE} that
+ * returns an object of the type {@link T}.
*
- * @param clazz
- * @param extensionClassName
+ * @param clazz A reference to {@link Class<T>}.
+ * @param extensionClassName The (fully-qualified) name of the class
+ * by which the extension shall be loaded.
*
* @return the instance of the extension class
+ *
+ * @throws ExtensionNotLoadedException Thrown if the load operation failed.
*/
// TODO: Throw custom exception if loading fails ...
Review Comment:
Wonder, if `ExtensionNotLoadedException` isn't the custom exception
mentioned in this TODO? The Javadoc states:
> Exception indicates that an OpenNLP extension could not be loaded.
##########
opennlp-tools/src/main/java/opennlp/tools/util/featuregen/BrownCluster.java:
##########
@@ -36,34 +36,40 @@
/**
*
* Class to load a Brown cluster document: word\tword_class\tprob
- * http://metaoptimize.com/projects/wordreprs/
- *
+ * <p>
+ * See: <a href="http://metaoptimize.com/projects/wordreprs/";>
Review Comment:
Link seems dead (at least for me). Getting a "Origin is unreachable Error
code 523" from cloudflare.
It isn't archived in the InternetArchive either. I was only able to find a
copy via `archive.ph`: https://archive.ph/p82oc
I think the related research paper is:
https://dl.acm.org/doi/10.5555/1858681.1858721
Maybe we can add a reference and mark the link as "broken" ?
> Enhance JavaDoc in opennlp.tools.util package
> ---------------------------------------------
>
> Key: OPENNLP-1413
> URL: https://issues.apache.org/jira/browse/OPENNLP-1413
> Project: OpenNLP
> Issue Type: Improvement
> Components: Documentation
> Affects Versions: 2.1.0
> Reporter: Martin Wiesner
> Assignee: Martin Wiesner
> Priority: Minor
> Fix For: 2.1.1
>
>
> The JavaDoc the _opennlp.tools.util_ 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)
