[CLN-list] [PATCH](master aka HEAD) Replace (some of) MAYBE_INLINEs with ISO C++ compliant equivalent.

Bruno Haible bruno at clisp.org
Fri Jan 11 00:22:23 CET 2008


Hello Richard,

> Ouch, that patch is ugly as sin.

I like Alexei's patch.
  - As I understand, it fixes the problem.
  - It does not introduce code duplication in the source code.

The only thing I would change about it is that I would use ##_inline
as a suffix, rather than inline_## as a prefix - because in CLN the
prefix is traditionally the number type (I_, RA_ etc.).

Yes, some comments about it are needed.

Bruno



More information about the CLN-list mailing list