Hi Jeffrey, > You should mail the comments directly to him: > > Jeffrey C Honig <[EMAIL PROTECTED]>
Done. > +/* > + * c_flags bits > + */ > +#define CF_TRUE (1<<0) /* usually means component is present */ > +#define CF_ADDRPARSED (2<<0) /* address has been parsed */ > +#define CF_DATEFAB (3<<0) /* datefield fabricated */ This looks wrong. Wouldn't 1, 2, and 4 be better values? Also, can FNORD be called repeatedly, i.e. does it insert a separator? It's definition didn't appear in the patch and I haven't the source to hand. Cheers, Ralph.