groovy-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jochen Theodorou (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (GROOVY-8200) Shorthand |= results in NPE
Date Wed, 24 May 2017 12:34:04 GMT

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

Jochen Theodorou commented on GROOVY-8200:
------------------------------------------

ah, ok, there is indeed a misunderstanding.  For {code:Java}
boolean x = true  
x & null
{code}
will NPE, but if you set x to false (the boolean sight on null) It does not NPE. That is imho
because not both sides are evaluated. This is wrong and should be fixed. The inconsistency
though is in the handling of & and &= not in the others. 

> Shorthand |= results in NPE
> ---------------------------
>
>                 Key: GROOVY-8200
>                 URL: https://issues.apache.org/jira/browse/GROOVY-8200
>             Project: Groovy
>          Issue Type: Bug
>    Affects Versions: 2.3.11
>         Environment: Groovy Version: 2.3.11 JVM: 1.7.0_80 Vendor: Oracle Corporation
OS: Linux
> java version "1.7.0_80"
> Java(TM) SE Runtime Environment (build 1.7.0_80-b15)
> Java HotSpot(TM) 64-Bit Server VM (build 24.80-b11, mixed mode)
>            Reporter: Daniel Hammer
>            Assignee: Jochen Theodorou
>         Attachments: shorthand_and.groovy, shorthand_or_symptom.groovy, shorthand_xor_symptom.groovy
>
>
> I've stumbled across some curious behaviour for the shorthand assignment operator in
Groovy v2.3.11 (symptom also present in Groovy v2.4.9).
> The following [boolean or logic|^shorthand_or_symptom.groovy], unexpectedly throws {{NullPointerException}}.
> {code}boolean x = null
> // short hand unrolled:  x = null || x
> x |= null // -> unexpected NullPointerException
> assert !x{code}
> The same approach for [boolean and logic|^shorthand_and.groovy], performs as expected.
> {code}boolean x = null
> // short hand unrolled:  x = null && x
> x &= null
> assert !x{code}
> It's possible to work around the issue with explicit cast.
> {code}x |= null as boolean{code}



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Mime
View raw message