On Wed, 2010-09-01 at 21:49 +0300, Johan Hedberg wrote: > Hi Bastien, > > On Wed, Sep 01, 2010, Bastien Nocera wrote: > > From: Bastien Nocera <hadess@xxxxxxxxxx> > > Date: Sun, 6 Jun 2010 15:48:26 +0100 > > Subject: [PATCH] (cups) Add ability to print IEEE1284 device ID > > > > Add ability to print IEEE1284 device ID for Bluetooth > > printers to allow auto-configuration once paired. > > --- > > cups/main.c | 83 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ > > 1 files changed, 83 insertions(+), 0 deletions(-) > > Thanks for the updated patch. It's now pushed upstream. Thanks. > > I've only fixed the style problems in the patch itself. I'm happy to do > > the rest of the file, if you have a "indent" magic incantation for it. > > In my experience indent can produce quite a mess compared to fixing > style issues manually. Anyway, I went ahead and did the cleanup myself > for the issues that I could quickly spot and pushed the fixes as a > separate patch upstream. OK. Indent should be able to do this without making a mess as long as you know the exact parameters. Mind, some of the restrictions feel like they're from COBOL days (79 characters line width?). I prefer readable code to mashed-up code that looks that way because of hard constraints. -- 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
