On 02/19/09 18:25, Glenn Faden wrote: > This looks good to me. It probably could have been coded differently > using getopt, but since that wasn't used in the original code, I think > your changes are acceptable.
I originally coded it using getopt, but that fails to handle "auths -ab username" correctly, as it can't tell if the "b" is the argument to "-a" or a (non-existant) option. This also meant it becomes tricky to handle "auths -asolaris.something username" correctly: the switch(argc) would fall to the wrong case (as argc==3 but it should fall to case 4). If there's an easy way to handle this I can swap out lines 82-92 for a getopt based command line parser (and update 93-99 to match). Bart
