Avoid the use of macros as Expected about gsl HOT 6 CLOSED

I would not label an issue as resolved when in reality it has been rejected.

from gsl.

The Expects() and Ensures() come from the guidelines. Ideally, we should have language support for them in form of contracts (see my CppCon talk). In the meantime, the fact that they are macros is an implementation detail, and should not be exposed as al uppercase letters.

from gsl.

Do you mean that C++17/20 will have Expects and Ensures as keywords?
Introducing these macros prevents a user having such names in their application to use GSL, which is bad IMHO.

But why do you use a macro when an inline function is enough? It is so hard to read

gsl::expects(x);
gsl::ensures(x);

?

from gsl.

Fair enough. Fixed.

from gsl.

I agree with @viboes here. I have 2 questions; 1. why is it a macro and 2. why does it not follow the coding standard of the rest of the library with regards to capitalization?

from gsl.

1. It is a macro so it can collect file/line information as helpful diagnostics when failures occur.
2. The capitalization follows the name suggested by the Core Guidelines. To be honest, that is something that does not particularly bother me either way. There is a standardization proposal to add a contract system to C++. That does use different capitalization so maybe it will change in time.

from gsl.