Subject: Re: Bluetooth update
To: None <firstname.lastname@example.org>
From: Christos Zoulas <email@example.com>
Date: 12/17/2005 23:17:02
In article <20051217210150.GA8187@panix.com>,
Thor Lancelot Simon <firstname.lastname@example.org> wrote:
>On Sat, Dec 17, 2005 at 08:54:33PM +0000, Iain Hibbert wrote:
>> 3. reading the style guide, it says use getopt (and presumably
>> getopt_long) - I didnt in btconfig, because I had it in mind to use
>> btconfig <device> pscan (enable pscan)
>> btconfig <device> -pscan (disable pscan)
>> but I am wavering on that...
>> btconfig --device <device> --pscan (enable pscan)
>> btconfig -d <device> --pscan=off (disable pscan)
>> might be preferable and is not difficult though not as easy to type (using
>> --pscan to enable seems counter intuitive to me :)
>Please do *not* use long options in new code for NetBSD.
>Can't you use getopt by skipping over the interface name by hand?
I think that btconfig should follow the syntax of ifconfig like
btconfig bt0 pscan
btconfig bt0 -pscan
From a cursory look at a couple of files the changes look fine
minus minor whitespace issues.