On Thu, 19 Apr 2007 19:23:26 -0400 Michael Wu <flamingice@xxxxxxxxxxxx> wrote: > On Thursday 19 April 2007 18:42, Stefano Brivio wrote: > > Fix coding style. > > > Looks mostly good except for.. > > > - // ehh? > > + // XXX: ehh? > Use FIXME if you want to put in a prefix. "XXX:" means we don't know if it's a FIXME, a TODO or just correct code. It's quite used. Unless this is a clear FIXME, I'd keep it. > > -} __attribute__ ((packed)); > > +} __attribute__((packed)); > I prefer the space here. Please see CodingStyle. -- Ciao Stefano - To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html