From owner-cvs-all Sun Mar 26 0:40:25 2000 Delivered-To: cvs-all@freebsd.org Received: from dustdevil.waterspout.com (dustdevil.waterspout.com [208.13.60.151]) by hub.freebsd.org (Postfix) with ESMTP id 2D94737B533; Sun, 26 Mar 2000 00:40:17 -0800 (PST) (envelope-from csg@dustdevil.waterspout.com) Received: (from csg@localhost) by dustdevil.waterspout.com (8.9.3/8.9.3) id DAA03506; Sun, 26 Mar 2000 03:42:24 -0500 (EST) (envelope-from csg) Date: Sun, 26 Mar 2000 03:42:24 -0500 From: "C. Stephen Gunn" To: Matt Dillon Cc: archie@FreeBSD.ORG, rwatson@FreeBSD.ORG, cvs-all@FreeBSD.ORG Subject: Re: cvs commit: src/sys/netinet if_ether.c Message-ID: <20000326034224.A3491@waterspout.com> References: <200003231859.KAA01229@freefall.freebsd.org> <20000326024700.A3342@waterspout.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-Mailer: Mutt 1.0.1i In-Reply-To: <20000326024700.A3342@waterspout.com>; from csg@waterspout.com on Sun, Mar 26, 2000 at 02:47:00AM -0500 Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.ORG On Sun, Mar 26, 2000 at 02:47:00AM -0500, C. Stephen Gunn wrote: > While I understand this fix. I copied this check from the "recommended" > procedure for calling m_pullup inside. There are several instances of > missing parenthesis in sys/netgraph/*.c as well. What I meant to say (since I was obviously out of my mind on caffiene): I understand the patch. While fixing the arpintr() problem, I read Archie's article on DaemonNews about Netgraph, and changed my m_pullup() call to match his example there. After seeing your correction to my code, I checked quickly in sys/netgraph/*.c and noticed several other calls to m_pullup without the parenthesis you added. I felt I should bring it up. That make much more sense, and actually resembles English. - Steve To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message