diff options
| author | Michael Karcher <flashrom@mkarcher.dialup.fu-berlin.de> | 2010-01-12 15:36:24 +0000 |
|---|---|---|
| committer | Michael Karcher <flashrom@mkarcher.dialup.fu-berlin.de> | 2010-01-12 15:36:24 +0000 |
| commit | e7f3209487b8adb681dd720f9ae512c8ded26f6e (patch) | |
| tree | 0de6c940c2288444810f571fabe0687c7e1ba40a /cli_classic.c | |
| parent | 9cce2f536581da4f278005a621ad95de1e21b284 (diff) | |
| download | flashrom-e7f3209487b8adb681dd720f9ae512c8ded26f6e.tar.gz flashrom-e7f3209487b8adb681dd720f9ae512c8ded26f6e.tar.bz2 flashrom-e7f3209487b8adb681dd720f9ae512c8ded26f6e.zip | |
Enable -Wshadow, clean code for that
This is not just for fun. We hit a real bug on BSD with the outl macros.
The macro variable tmp collided with the tmp from outer scope.
second revision, now also taking care of inb/inw/inl. While that shadowing
did not introduce bugs (yet), of course it breaks the build on BSD when
-Wshadow is enabled.
Corresponding to flashrom svn r860.
Signed-off-by: Michael Karcher <flashrom@mkarcher.dialup.fu-berlin.de>
Acked-by: Luc Verhaegen <libv@skynet.be>
Diffstat (limited to 'cli_classic.c')
| -rw-r--r-- | cli_classic.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/cli_classic.c b/cli_classic.c index d3e7a159e..6cb0578f0 100644 --- a/cli_classic.c +++ b/cli_classic.c @@ -150,7 +150,6 @@ int cli_classic(int argc, char *argv[]) if (argc > 1) { /* Yes, print them. */ - int i; printf_debug("The arguments are:\n"); for (i = 1; i < argc; ++i) printf_debug("%s\n", argv[i]); |