You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Dmitri Blinov (JIRA)" <ji...@apache.org> on 2016/02/01 13:06:39 UTC
[jira] [Created] (JEXL-189) Possible bug in
Interpreter.isCancelled()
Dmitri Blinov created JEXL-189:
----------------------------------
Summary: Possible bug in Interpreter.isCancelled()
Key: JEXL-189
URL: https://issues.apache.org/jira/browse/JEXL-189
Project: Commons JEXL
Issue Type: Bug
Affects Versions: 3.0
Reporter: Dmitri Blinov
Priority: Trivial
Reported by FindBugs is the message:
"Questionable use of non-short-circuit logic in org.apache.commons.jexl3.internal.Interpreter.isCancelled()"
{code}
protected boolean isCancelled() {
if (cancelled | Thread.interrupted()) {
cancelled = true;
}
return cancelled;
}
{\code}
Maybe it is better to replace method as
{code}
protected boolean isCancelled() {
if (Thread.interrupted())
cancelled = true;
return cancelled;
}
{\code}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)