[
https://issues.apache.org/jira/browse/NUTCH-2475?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16735331#comment-16735331
]
Hudson commented on NUTCH-2475:
-------------------------------
SUCCESS: Integrated in Jenkins build Nutch-trunk #3598 (See
[https://builds.apache.org/job/Nutch-trunk/3598/])
NUTCH-2475 If and else-if branches has the same condition - remove (snagel:
[https://github.com/apache/nutch/commit/d0a4abfd7cd294d7ffb14b6947c243e46c65f265])
* (edit)
src/plugin/protocol-ftp/src/java/org/apache/nutch/protocol/ftp/Client.java
> If and else-if branches has the same condition
> ----------------------------------------------
>
> Key: NUTCH-2475
> URL: https://issues.apache.org/jira/browse/NUTCH-2475
> Project: Nutch
> Issue Type: Bug
> Components: plugin, protocol
> Affects Versions: 2.2, 1.14
> Reporter: songwanging
> Assignee: Sebastian Nagel
> Priority: Minor
> Fix For: 2.4, 1.16
>
>
> Our tool DeepTect has detected a piece of buggy code snippet, in which the if
> and else branches has the same condition.
> Path:
> nutch/src/plugin/protocol-ftp/src/java/org/apache/nutch/protocol/ftp/Client.java
>
> {code:java}
> private boolean _notBadReply(int reply) {
> if (FTPReply.isPositiveCompletion(reply)) {
> // do nothing
> ...
> } else if (reply == 451) { // FTPReply.ACTION_ABORTED
> // some ftp servers reply 451, e.g.,
> // ProFTPD [ftp.kernel.org]
> // there is second reply witing? no!
> // getReply();
> } else if (reply == 451) { // FTPReply.ACTION_ABORTED
> } else {
> // what other kind of ftp server out there?
> return false;
> }
> return true;
> }
> {code}
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
