Date: Mon, 14 Jan 2013 14:47:09 -0800 From: Michael Sierchio <kudzu@tenebras.com> To: Alfred Perlstein <bright@mu.org> Cc: freebsd-net@freebsd.org, John Baldwin <jhb@freebsd.org>, net@freebsd.org Subject: Re: [PATCH] Don't imply TCP and UDP socket options are bitmasks Message-ID: <CAHu1Y72d4O=pnT9A61PAmEJXSfQ2SBCQavQw5hnnWS6Nwe-2jw@mail.gmail.com> In-Reply-To: <50F483E8.2040107@mu.org> References: <201301141550.13577.jhb@freebsd.org> <50F47BB8.9000409@mu.org> <201301141656.37175.jhb@freebsd.org> <50F483E8.2040107@mu.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Change "Don't imply TCP and UDP socket options are bitmasks" to "Don't infer TCP and UDP socket options are bitmasks" - M On Mon, Jan 14, 2013 at 2:17 PM, Alfred Perlstein <bright@mu.org> wrote: > On 1/14/13 4:56 PM, John Baldwin wrote: >> >> On Monday, January 14, 2013 4:42:16 pm Alfred Perlstein wrote: >>> >>> Wouldn't a comment over the code suffice? >>> >>> Something like your email as a header would actually work very nicely! >>> >>> I think just using decimal would be more confusing than explicitly >>> calling it out like: >>> >>> /* begin enumerated (not bitmask) socket option specifiers */ >>> #define TCP_MAXSEG 0x02 /* set maximum segment size */ >>> #define TCP_NOPUSH 0x04 /* don't push last block of write */ >>> #define TCP_NOOPT 0x08 /* don't use TCP options */ >>> #define TCP_MD5SIG 0x10 /* use MD5 digests (RFC2385) */ >>> /* end enumerated socket option specifiers */ >> >> I have a patch I'll post next which will add a new option as '3'. I think >> that >> will make it more obvious and avoid having new options follow the old >> pattern. >> > Any objection to adding the contents of that email as a comment section? It > really would help. > > > -Alfred > > _______________________________________________ > freebsd-net@freebsd.org mailing list > http://lists.freebsd.org/mailman/listinfo/freebsd-net > To unsubscribe, send any mail to "freebsd-net-unsubscribe@freebsd.org"
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAHu1Y72d4O=pnT9A61PAmEJXSfQ2SBCQavQw5hnnWS6Nwe-2jw>