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

Reply via email to