ACK: [PATCH] acpi: dscdump: reduce scope of loop counter i

[ivanhu]

On 6/11/19 5:05 PM, Colin King wrote:
> From: Colin Ian King <colin.king at canonical.com>
>
> Clean up a cppcheck style check.
>
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
>  src/acpi/dsddump/dsddump.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/acpi/dsddump/dsddump.c b/src/acpi/dsddump/dsddump.c
> index 1e3c26c1..c4e0610b 100644
> --- a/src/acpi/dsddump/dsddump.c
> +++ b/src/acpi/dsddump/dsddump.c
> @@ -144,7 +144,6 @@ static int dsddump_test1(fwts_framework *fw)
>  			ACPI_OBJECT_LIST arg_list;
>  			ACPI_OBJECT *obj;
>  			ACPI_BUFFER buf;
> -			uint32_t i;
>  			int ret;
>  
>  			arg_list.Count   = 0;
> @@ -161,6 +160,7 @@ static int dsddump_test1(fwts_framework *fw)
>  			    (obj->Package.Elements[0].Type == ACPI_TYPE_BUFFER) &&
>  			    (obj->Package.Elements[0].Buffer.Pointer != NULL) &&
>  			    (obj->Package.Elements[0].Buffer.Length == 16)) {
> +				uint32_t i;
>  
>  				fwts_log_info_verbatim(fw, "Name: %s", name);
>  				for (i = 0; i < obj->Package.Count; i += 2)


Acked-by: Ivan Hu <ivan.hu at canonical.com>


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: OpenPGP digital signature
URL: <https://lists.ubuntu.com/archives/fwts-devel/attachments/20190612/2592c3a6/attachment.sig>