Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix RETNIL macro for code #577

Merged
merged 1 commit into from
Jan 17, 2017
Merged

Fix RETNIL macro for code #577

merged 1 commit into from
Jan 17, 2017

Conversation

PabstMirror
Copy link
Contributor

#576 broke TRACE macros such as
TRACE_2("",_unit,local _unit); - due to isNil "local _unit" is true.
and
TRACE_1("",QUOTE(ADDON)); - quotation problems

Using the {} version of isNil seems to work just as well and fixes these problems.

@PabstMirror PabstMirror added this to the 3.2 milestone Jan 17, 2017
@commy2
Copy link
Contributor

commy2 commented Jan 17, 2017

BEEP. Reminder to myself to mute the machine when running it overnight.

This is right. This is how it was before we screwed it up by trying to use param. Thanks.

@commy2 commy2 merged commit 6d462b8 into master Jan 17, 2017
@commy2 commy2 deleted the fixRetNil branch January 17, 2017 03:39
@654wak654
Copy link
Contributor

😢 whoops

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants