ACK: [PATCH 02/26] lib: fwts_acpi_method: fix for loop index type
Alex Hung
alex.hung at canonical.com
Mon Oct 15 02:44:05 UTC 2012
On 10/15/2012 04:31 AM, Colin King wrote:
> From: Colin Ian King <colin.king at canonical.com>
>
> Fix the type of a for loop to match that of the package Count type
>
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
> src/lib/src/fwts_acpi_method.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/lib/src/fwts_acpi_method.c b/src/lib/src/fwts_acpi_method.c
> index 4a0e7ec..7974545 100644
> --- a/src/lib/src/fwts_acpi_method.c
> +++ b/src/lib/src/fwts_acpi_method.c
> @@ -182,7 +182,7 @@ char *fwts_method_exists(char *name)
> */
> static void fwts_method_dump_object_recursive(fwts_framework *fw, ACPI_OBJECT *obj, int depth, int index)
> {
> - int i;
> + uint32_t i;
> char index_buf[5];
>
> if (index > -1)
> @@ -206,7 +206,7 @@ static void fwts_method_dump_object_recursive(fwts_framework *fw, ACPI_OBJECT *o
> case ACPI_TYPE_PACKAGE:
> fwts_log_info_verbatum(fw, "%*s%sPackage has %d elements:",depth * 2, "",
> index_buf, obj->Package.Count);
> - for (i=0; i < obj->Package.Count; i++) {
> + for (i = 0; i < obj->Package.Count; i++) {
> ACPI_OBJECT *element = &obj->Package.Elements[i];
> fwts_method_dump_object_recursive(fw, element, depth + 1, i);
> }
>
Acked-by: Alex Hung <alex.hung at canonical.com>
More information about the fwts-devel
mailing list