On Mon, Jun 8, 2015 at 11:33 PM, hch@xxxxxx <hch@xxxxxx> wrote: > On Wed, Jun 03, 2015 at 07:24:34PM +0000, Williams, Dan J wrote: >> > > +static inline struct acpi_nfit_memory_map *__to_nfit_memdev(struct nfit_mem *nfit_mem) >> > >> > This line is over 80 characters. >> >> I generally don't see the point of fixing up occasional small incursions >> over 80 characters if it makes the code more amenable to grep. > > Just use shorter type names or add a line break before the symbol > names. Too long lines are highly annoying. Note that "acpi_nfit_memory_map" is the identifier chosen by ACPICA, I'm just a consumer. That said I've gone and done a 80 column cleanup across all the patches. -- To unsubscribe from this list: send the line "unsubscribe linux-api" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html