Hi Bastien, On Tue, Jun 08, 2010, Bastien Nocera wrote: > > I have nothing against pushing the patch upstream as long as its coding > > style issues are fixed: > <snip> > > Mixed tabs and spaces. > > You do realise that all those are cut'n'paste from another function in > the same source file? Nope, didn't realize that. So the whole file needs coding style cleanups then. > > > + fprintf(stderr, "Invalid Bluetooth address '%s'\n", argv[2]); > > > > Too long line. > > Right, I'll fix that. Thanks. Johan -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html