Issues with includes about re HOT 5 CLOSED

hi,

your applications Makefile should include re.mk to get the same CFLAGS etc as re,
see the redemo project:

https://github.com/creytiv/redemo/blob/master/Makefile#L15

it is recommended that the application only includes <re.h>, see redemo:

https://github.com/creytiv/redemo/blob/master/src/sip_ua/sip_ua.c#L21

the redefinition of types is only a fallback solution, in case inttypes.h is not available.
make sure HAVE_INTTYPES_H is defined in your application.

either in your makefile or in your code:

#define HAVE_INTTYPES_H 1
#include <re.h>

Alfred

from re.

Hi Alfred,

thanks for the response, it helped a lot. With HAVE_INTTYPES_H and HAVE_STDBOOL_H it now almost works, although this part in re_types.h causes inline redefinition issue:

/* Needed for MS compiler */
#ifdef _MSC_VER
#define inline _inline
#endif

I believe that definition is needed in MS compilers only when compiled as C, but it causes following issue when compiled as C++: zeromq/czmq#212. The guard if (!defined (__cplusplus) may be needed. After fixing this single thing, everything works perfectly for me.

from re.

Would you like to submit a Pull Request for that change (inline and C++) ?

It would be great to test the change with different MS compilers.
(I dont have access to Windows machines).

from re.

Created the pull request, I have tested it only on latest Visual Studio 2017 ( I do not have older versions at the moment), but from what I remember, we have used C++ inline keyword for years (even in Visual Studio 6 which is like from 1998), and it worked fine, and for C it now works like before.

Also by the way, I mention it in the pull request comments, but the inline was not available for C because of poor C99 support, which is not the case anymore, so basically in new version you won't need the inline keywoard define at all, but it works with it as well.

from re.

fixed in commit f9224f3

Thanks @TheSil !

from re.