On Tue, Feb 17, 2009 at 10:24:28PM +0100, Otto Moerbeek wrote:
quoted text
> This diff is very wrong. To know why, read the fgetln(3) manpage.

OK, but it would be handy to allow comments. The attached patch honors
fgetln(3) habit not to return C style strings. But it does not detect
empty lines, i.e. lines with len==0.
--- grey.c.orig	Tue Feb 17 23:27:39 2009
+++ grey.c	Tue Feb 17 23:30:57 2009
@@ -321,6 +321,8 @@
 		while ((buf = fgetln(fp, &len))) {
 			if (buf[len-1] == '\n')
 				len--;
+			if ( len && (buf[0]=='#') )
+				continue;
 			if ((m = malloc(sizeof(struct mail_addr))) == NULL)
 				goto bad;
 			if ((len + 1) > sizeof(m->addr)) {