Patch set 3:Code-Review -1
View Change
1 comment:
File bitbang_spi.c:
Patch Set #3, Line 19: #include <ctype.h>
Uh, do we want global headers above local headers?
No, but I think this can be fixed by separating the global headers and local headers with a newline. Here is the documentation for the #include sorting option:
# Whether to sort consecutive single-line '#include' statements (C/C++) and
# '#import' statements (Objective-C). Be aware that this has the potential to
# break your code if your includes/imports have ordering dependencies.
mod_sort_include = false # true/false
To view, visit change 42556. To unsubscribe, or for help writing mail filters, visit settings.
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: Ib6bd3ab49e841768f07c2705f71be66e9308bbca
Gerrit-Change-Number: 42556
Gerrit-PatchSet: 3
Gerrit-Owner: David Hendricks <david.hendricks@gmail.com>
Gerrit-Reviewer: David Hendricks <david.hendricks@gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Angel Pons <th3fanbus@gmail.com>
Gerrit-CC: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-CC: Vadim Bendebury <vbendeb@chromium.org>
Gerrit-Comment-Date: Wed, 24 Jun 2020 16:12:19 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: Angel Pons <th3fanbus@gmail.com>
Gerrit-MessageType: comment