diff options
author | Harvey Harrison <[email protected]> | 2012-10-13 18:07:27 -0700 |
---|---|---|
committer | Harvey Harrison <[email protected]> | 2012-10-13 18:09:45 -0700 |
commit | 6838d3593512d0388948c784023303d966b6e298 (patch) | |
tree | 0a379d9f42728d8ecaf66c0dca892005b8a39b9d | |
parent | 542e7c364ced2865c5deac1498f5ff537a31c0b8 (diff) |
gluegen: simplify expression comparing against Boolean.TRUE
As a last resort we can just return the boolean value directly here, also no
need to try/catch as valueOf does not throw any exceptions.
Pointed out by Findbugs.
Signed-off-by: Harvey Harrison <[email protected]>
-rw-r--r-- | src/java/com/jogamp/gluegen/pcpp/PCPP.java | 9 |
1 files changed, 2 insertions, 7 deletions
diff --git a/src/java/com/jogamp/gluegen/pcpp/PCPP.java b/src/java/com/jogamp/gluegen/pcpp/PCPP.java index 07c7bd3..30d8dd8 100644 --- a/src/java/com/jogamp/gluegen/pcpp/PCPP.java +++ b/src/java/com/jogamp/gluegen/pcpp/PCPP.java @@ -961,13 +961,8 @@ public class PCPP { // ok, it's not a valid decimal value, try hex/octal value return Long.parseLong(word) != 0; } catch (NumberFormatException nfe2) { - try { - // ok, it's not a valid hex/octal value, try boolean - return Boolean.valueOf(word) == Boolean.TRUE; - } catch (NumberFormatException nfe3) { - // give up; the symbol isn't a numeric or boolean value - return false; - } + // ok, it's not a valid hex/octal value, try boolean last + return Boolean.valueOf(word).booleanValue(); } } } |