[Issue 9450] make assert an implicit "version (assert)"

d-bugmail at puremagic.com d-bugmail at puremagic.com
Mon Feb 4 10:05:58 PST 2013


http://d.puremagic.com/issues/show_bug.cgi?id=9450


Andrej Mitrovic <andrej.mitrovich at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |andrej.mitrovich at gmail.com,
                   |                            |k.hara.pg at gmail.com


--- Comment #1 from Andrej Mitrovic <andrej.mitrovich at gmail.com> 2013-02-04 10:05:57 PST ---
CC'ed Kenji. 

@Kenji: I think all that's required here is to return early in
'AssertExp::semantic' if 'global.params.release' is true, e.g.:

Expression *AssertExp::semantic(Scope *sc)
{
    if (global.params.release)
    {
        type = Type::tvoid;
        return this;
    }

    // ...
}

It seems to work for this simple case. Would this be ok?

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------


More information about the Digitalmars-d-bugs mailing list