On 17 October 2016 at 16:17, Simon Glass <s...@chromium.org> wrote: > On 16 October 2016 at 09:13, Stefan Brüns <stefan.bru...@rwth-aachen.de> > wrote: >> GCC 6.2 reasonably complains about the current code: >> >> ../cmd/tpm_test.c: In function ‘do_tpmtest’: >> ../cmd/tpm_test.c:540:3: warning: this ‘for’ clause does not guard... >> [-Wmisleading-indentation] >> for (i = 0; i < argc; i++) >> ^~~ >> ../cmd/tpm_test.c:542:4: note: ...this statement, but the latter is >> misleadingly indented as if it is guarded by the ‘for’ >> printf("\n------\n"); >> ^~~~~~ >> >> Signed-off-by: Stefan Brüns <stefan.bru...@rwth-aachen.de> >> --- >> cmd/tpm_test.c | 11 +++++------ >> 1 file changed, 5 insertions(+), 6 deletions(-) > > Reviewed-by: Simon Glass <s...@chromium.org>
Updated to drop the C99 variable decl. Applied to u-boot-dm/next, thanks! _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot