This is something that many people feel very strongly about and we never really agreed on. Feel free to reopen the discussion if you want, but can we please revert this patch in the meantime?

I would prefer not to revert. No matter the reasoning
and latest discussions, this brings the configuration
closer to our coding style (that was copied from the
Linux kernel years ago). It says you may exceed the
80 chars limit if it increases readability.

So if the 80 chars aren't a hard limit, I don't see
why we should let tools bother us with it.

View Change

To view, visit change 31651. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I9293a69d1bea900da36501cde512004d0695ad37
Gerrit-Change-Number: 31651
Gerrit-PatchSet: 2
Gerrit-Owner: Patrick Georgi <pgeorgi@google.com>
Gerrit-Reviewer: Martin Roth <martinroth@google.com>
Gerrit-Reviewer: Nico Huber <nico.h@gmx.de>
Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Julius Werner <jwerner@chromium.org>
Gerrit-CC: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-Comment-Date: Fri, 08 Mar 2019 22:37:43 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment