On 14 Apr 2016 09:45, Yury Gribov wrote: > -extern void using_history PARAMS((void)); > +extern READLINE_EXPORT void using_history PARAMS((void));
since READLINE_EXPORT must be defined regardless, why not fold "extern"
into it too ?
> --- a/rlstdc.h
> +++ b/rlstdc.h
> @@ -54,4 +54,8 @@
> # endif
> #endif
>
> +#if defined(__GNUC__) && (__GNUC__ >= 4) && !defined(__CYGWIN__) &&
> !defined(__MINGW32__)
> +#define READLINE_EXPORT __attribute__((visibility("default")))
> +#endif
it seems you forgot to define READLINE_EXPORT in the negative case,
so all the builds go boom :).
that said, what's with the extensive cpp checks ? you already did a
configure test for it, so you should just re-use that logic and have
it set something up in config.h or add to CPPFLAGS.
-mike
signature.asc
Description: Digital signature
_______________________________________________ Bug-readline mailing list [email protected] https://lists.gnu.org/mailman/listinfo/bug-readline
