From owner-freebsd-net Thu Aug 16 1:33:45 2001 Delivered-To: freebsd-net@freebsd.org Received: from mailman.zeta.org.au (mailman.zeta.org.au [203.26.10.16]) by hub.freebsd.org (Postfix) with ESMTP id 6FE9E37B40F for ; Thu, 16 Aug 2001 01:33:39 -0700 (PDT) (envelope-from bde@zeta.org.au) Received: from bde.zeta.org.au (bde.zeta.org.au [203.2.228.102]) by mailman.zeta.org.au (8.9.3/8.8.7) with ESMTP id SAA12896; Thu, 16 Aug 2001 18:33:30 +1000 Date: Thu, 16 Aug 2001 18:33:29 +1000 (EST) From: Bruce Evans X-X-Sender: To: Julian Elischer Cc: Subject: Re: IPV6/KAME/protosw integration cleanup In-Reply-To: Message-ID: <20010816180314.P27152-100000@besplex.bde.org> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-freebsd-net@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.org > I will not be doing any changes that affect them, Though I may still add > the prototype definitions in protsw.h as that's a generally useful thing > to do. I think those are the least useful of your changes. They are certainly the most intrusive if the function typedefs are actually used. The corresponding function typedefs for the cdevsw functions are even less useful since all the cdevsw's are now static and the prototypes can't be used in function bodies (except for variants which give the args in a list like SYSCTL_HANDLER_ARGS, but don't go there). In most cases, forward declarations of the cdevsw functions aren't necessary provided we move the cdevsw data declarations after all the function bodies. I normally dislike shuffling code to avoid forward declarations, and like to forward-declare even static functions that are defined before they are used (since this gives an easy to find list of the functions and doesn't depend on the order of the function bodies), but the cdevsw data declaration already provides a lot of structure including a list of all the cdevsw functions, so I don't mind making a special case for it. It's normal to use a similar hack for modules (put DRIVER_MODULE() declarators at the end). Protosw's are still externed, so more declarations are needed, but that should change. Bruce To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-net" in the body of the message