Subject: Re: Bluetooth update
To: None <tech-kern@netbsd.org>
From: Christos Zoulas <christos@astron.com>
List: tech-kern
Date: 12/17/2005 23:17:02
In article <20051217210150.GA8187@panix.com>,
Thor Lancelot Simon <tls@rek.tjls.com> 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?
>
>Thor
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.
christos