Skip to content

Remove the useless condition. - #74

Closed
Kui-Liu wants to merge 1 commit into
apache:trunkfrom
Kui-Liu:master3
Closed

Kui-Liu wants to merge 1 commit into
apache:trunkfrom
Kui-Liu:master3

Conversation

@Kui-Liu

@Kui-Liu Kui-Liu commented Oct 12, 2017

Copy link
Copy Markdown

The value of the condition "numChars < 0" has been narrowed by a previous IfStatement.
It has no effect on the current control flow.
http://findbugs.sourceforge.net/bugDescriptions.html#UC_USELESS_CONDITION

The value of the condition "numChars < 0" has been narrowed by a previous IfStatement.
It has no effect on the current control flow.
http://findbugs.sourceforge.net/bugDescriptions.html#UC_USELESS_CONDITION
@asfgit

asfgit commented Oct 12, 2017

Copy link
Copy Markdown

Can one of the admins verify this patch?

@asfgit asfgit closed this in bc7d79a Oct 13, 2017
Alain-Bearez pushed a commit to cuali/poi that referenced this pull request Dec 12, 2019
…maller pull requests.

This closes apache#74, This closes apache#75, This closes apache#76

git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1812097 13f79535-47bb-0310-9956-ffa450edef68
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants