warning C4002: too many actual parameters for macro 'assert'

Hi,

This compiles:

Code:

double a = cmath::euclideanDistance2D<BlasVector,double>(mTargetPosition, mrEntity.vehicleModel().pos());

assert(10.0 < a);

But for this

Code:

assert(10.0 < cmath::euclideanDistance2D<BlasVector,double>(mTargetPosition, mrEntity.vehicleModel().pos()));

msvc gives these erros and warnings:

Code:

1>.\GoalMoveXY.cpp(46) : warning C4002: too many actual parameters for macro 'assert'
1>.\GoalMoveXY.cpp(46) : error C2143: syntax error : missing ',' before ')'
1>.\GoalMoveXY.cpp(46) : error C2059: syntax error : ')'
1>.\GoalMoveXY.cpp(46) : error C2143: syntax error : missing ',' before ')'
1>.\GoalMoveXY.cpp(46) : error C2059: syntax error : ')'
1>.\GoalMoveXY.cpp(50) : error C2143: syntax error : missing ',' before ')'
1>.\GoalMoveXY.cpp(53) : error C2065: 'way' : undeclared identifier
1>.\GoalMoveXY.cpp(53) : error C2228: left of '.rend' must have class/struct/union
1>        type is ''unknown-type''
1>.\GoalMoveXY.cpp(53) : error C2143: syntax error : missing ',' before ')'
1>.\GoalMoveXY.cpp(56) : error C2143: syntax error : missing ';' before '}'
1>.\GoalMoveXY.cpp(60) : error C2143: syntax error : missing ';' before '{'
1>.\GoalMoveXY.cpp(65) : error C2143: syntax error : missing ';' before '{'
1>.\GoalMoveXY.cpp(68) : error C2143: syntax error : missing ';' before '}'
1>.\GoalMoveXY.cpp(70) : error C2143: syntax error : missing ';' before '{'
1>.\GoalMoveXY.cpp(71) : error C2143: syntax error : missing ',' before ')'
1>.\GoalMoveXY.cpp(72) : error C2143: syntax error : missing ',' before ')'
1>.\GoalMoveXY.cpp(81) : error C2143: syntax error : missing ';' before '{'
1>.\GoalMoveXY.cpp(83) : error C2143: syntax error : missing ';' before '}'
1>.\GoalMoveXY.cpp(84) : error C2143: syntax error : missing ';' before '}'
1>.\GoalMoveXY.cpp(87) : error C2143: syntax error : missing ';' before '}'
1>.\GoalMoveXY.cpp(90) : error C2143: syntax error : missing ';' before '}'
1>.\GoalMoveXY.cpp(91) : error C2143: syntax error : missing ';' before '}'
1>.\GoalMoveXY.cpp(91) : fatal error C1004: unexpected end-of-file found

Arn't template parameters allowed inside macro calls?

Thank you!

They probably are, but perhaps you need an extra set of paranthesis, as the preprocessor has no idea what a template is, and will just split arguments by commas.

--
Mats

More precisely, it doesn't recognize < > as comma-masking. So yes, you need an extra pair of parenthesis.

Originally Posted by CornedBee

More precisely, it doesn't recognize < > as comma-masking. So yes, you need an extra pair of parenthesis.

Yes, but the C preprocessor knows nothing ELSE about templates either, does it?

--
Mats

But it doesn't need to know anything else. It's the commas that get you in trouble.

Originally Posted by CornedBee

But it doesn't need to know anything else. It's the commas that get you in trouble.

You are right - there is no meaning to template that would matter to the preprocessor. I was trying (hard) to come up with some other obscure case where template "knowledge" would make a difference, but the only place which matters is commas in the <> section of templates.

--
Mats

I guess the preprocessor sees it as:

Code:

assert(x < y < z,double>(mTargetPosition, mrEntity.vehicleModel().pos()));

Yes, something like that. There are very few things in the syntax that matters to the preprocessor. It knows about whitespaces, comma, lparen '(' and rparen ')', the # sign, and that's about it. The rest of it is about text replacement of given parameters.

--
Mats

Thank you all. Another pair of parenthesis did the trick.

Strings. It knows about strings.

Originally Posted by CornedBee

Strings. It knows about strings.

Yes, of course. As in "somestuff", not std::strings

--
Mats

In case you want to "protect" a comma and parenthesis don't work (for example with templates), you can use the following macro:

Code:

#define CONCAT(...) __VA_ARGS__
#define ID(x) x
//...
printf("%s %s!\n", ID(CONCAT("hello", "world")) );

See more on my website...

HTH,
Ingo

O_o

Did you actually just search for questions to answer in order to advertise your site as a pathetic attempt to circumvent the rules?

Anyway, this is the C++ forum--not the C99 forum.

Soma

Originally Posted by phantomotap

O_o

Did you actually just search for questions to answer in order to advertise your site as a pathetic attempt to circumvent the rules?

Anyway, this is the C++ forum--not the C99 forum.

Soma

lol my thought too

Link deleted, thread closed. Don't resurrect old threads, especially in order to promote your site.