Pequeño parche para BlueZ/hcitool
Hace más de un mes trabajando con hcitool detecté lo que a mi juicio era un bug muy tonto y de fácil solución, así que me puse a ello, hice un parche y lo envié a los desarrolladores de BlueZ…
Copy+paste del mensaje que envié a la lista bluez-devel:
Hi! If I want to scan on hci0 the next sintax is bad, but hcitool doesn't complain: pablo@golgi:~$ hcitool hci0 scan pablo@golgi:~$ (I better should use hcitool -i hci0 scan) I've written a little patch to warn when an unrecognised command is given: pablo@golgi:~/work/hcitool-patch1/utils/tools$ diff -u hcitool.c.orig hcitool.c
--- hcitool.c.orig 2007-12-28 20:06:58.000000000 +0100
+++ hcitool.c 2007-12-29 11:35:00.000000000 +0100
@@ -2329,5 +2329,11 @@
command[i].func(dev_id, argc, argv);
break;
}
+
+ if(!command[i].cmd) {
+ fprintf(stderr, "\"%s\" isn't a valid command. Try --help for a list
of commands.\n", argv[0]);
+ exit(1);
+ }
+
return 0;
}
Now, when I ask for an inexistent command it complains:
pablo@golgi:~$ ./hcitool nop scan
"nop" isn't a valid command. Try --help for a list of commands.
pablo@golgi:~$
If you find it useful, feel free to commit it.
Bye.
La verdad es que el parche es tontísimo y no sé si lo aplicarán, pero hay queda, por si a alguien le vale para algo. Me dio más guerra seguir el coding style que hacer el propio parche (por aquí va el hilo).
Update: me acabo de fijar que me dieron más indicaciones para el coding style de las que seguí (no las vi), razón adicional para omitir el parche.
