commit 7d0e3ba5497f778636001e91a775c07f9fe565c4
author Justin Chadwell <justin.chadwell@arm.com> Tue Sep 17 15:21:50 2019 +0100
committer Madhukar Pappireddy <madhukar.pappireddy@arm.com> Tue Nov 19 08:53:16 2019 -0600
tree fcca63549f0f8c7ab15e706c6995de38aa2290d0
parent 80e264b03a2db32719628c52be676bfd74b0448d

Enable -Wshadow always

Variable shadowing is, according to the C standard, permitted and valid
behaviour. However, allowing a local variable to take the same name as a
global one can cause confusion and can make refactoring and bug hunting
more difficult.

This patch moves -Wshadow from WARNING2 into the general warning group
so it is always used. It also fixes all warnings that this introduces
by simply renaming the local variable to a new name

Change-Id: I6b71bdce6580c6e58b5e0b41e4704ab0aa38576e
Signed-off-by: Justin Chadwell <justin.chadwell@arm.com>