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

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

mawiesne commented on PR #1:
URL: https://github.com/apache/opennlp-sandbox/pull/1#issuecomment-1427000580

   @kinow Could you resolve the conflict so we can re-evaluate if this PR holds 
all CI steps and can be merged safely?




> A condition that is always true.
> --------------------------------
>
>                 Key: OPENNLP-985
>                 URL: https://issues.apache.org/jira/browse/OPENNLP-985
>             Project: OpenNLP
>          Issue Type: Bug
>          Components: wsd
>            Reporter: JC
>            Priority: Trivial
>
> I've found a code smell or typo in a recent github snapshot. (opennlp-snadbox)
> Path: 
> opennlp-wsd/src/main/java/opennlp/tools/disambiguator/datareader/Paragraph.java
> {code:java}
> 85   public boolean contains(String wordTag) {
> 86 
> 87     for (Sentence isentence : this.getSsentences()) {
> 88       for (Word iword : isentence.getIwords()) {
> 89         if (iword.equals(iword))
> 90           return true;
> 91       }
> 92     }
> 93 
> 94     return false;
> 95   }
> {code}
> Line 89 is always true. This might be a trivial issue but wanted to report 
> just in case. Thanks!



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

Reply via email to