<p><br>
On May 25, 2012 4:44 PM, "Mike Kelly" <<a href="mailto:pioto@pioto.org">pioto@pioto.org</a>> wrote:<br>
><br>
> ---<br>
>  test/smtp-dummy.c |    5 ++++-<br>
>  1 file changed, 4 insertions(+), 1 deletion(-)<br>
><br>
> diff --git a/test/smtp-dummy.c b/test/smtp-dummy.c<br>
> index 3801a5e..aa82fa1 100644<br>
> --- a/test/smtp-dummy.c<br>
> +++ b/test/smtp-dummy.c<br>
> @@ -33,11 +33,14 @@<br>
>  * have been warned.<br>
>  */<br>
><br>
> +#define _GNU_SOURCE /* for getline */</p>
<p>First of all, thanks for sharing your work.</p>
<p>Independent of this patch, I've been thinking about defining _GNU_SOURCE in the makefile. I don't think it's very elegant to #define it everywhere. It would also pave the way towards building with -pedantic (and possibly -std=c99) in a portable way. For example, the compat test code #defines _GNU_SOURCE while the actual code does not, which I think is an error.</p>

<p>I'll look into this after the weekend (unless you beat me to it).</p>
<p>BR,<br>
Jani.<br></p>
<p>>  #include <stdio.h><br>
>  #include <stdlib.h><br>
>  #include <string.h><br>
>  #include <errno.h><br>
> -#include <netinet/ip.h><br>
> +#include <sys/types.h><br>
> +#include <sys/socket.h><br>
> +#include <netinet/in.h><br>
>  #include <netdb.h><br>
>  #include <unistd.h><br>
><br>
> --<br>
> 1.7.10.2<br>
><br>
> _______________________________________________<br>
> notmuch mailing list<br>
> <a href="mailto:notmuch@notmuchmail.org">notmuch@notmuchmail.org</a><br>
> <a href="http://notmuchmail.org/mailman/listinfo/notmuch">http://notmuchmail.org/mailman/listinfo/notmuch</a></p>