Skip to content

#if defined(DEBUG) && DEBUG compiling error#380

Open
godera wants to merge 1 commit into
ccgus:masterfrom
godera:master
Open

#if defined(DEBUG) && DEBUG compiling error#380
godera wants to merge 1 commit into
ccgus:masterfrom
godera:master

Conversation

@godera
Copy link
Copy Markdown

@godera godera commented Jun 8, 2015

replaced by #ifdef DEBUG

@ccgus
Copy link
Copy Markdown
Owner

ccgus commented Jun 17, 2015

What's the compiler error?

@phoney
Copy link
Copy Markdown

phoney commented Oct 17, 2015

Actually, IMO both are wrong. Proper use of DEBUG should be
#if DEBUG
This avoids the problem of #define DEBUG 0
Having said that, the proposed fix doesn't fix any compiler warnings that I can see.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants