ACK: [PATCH] acpica: fwts_iasl_interface: reduce scope of variable 'i'

ivanhu ivan.hu at canonical.com
Thu Aug 18 02:40:37 UTC 2016



On 2016年07月26日 04:56, Colin King wrote:
> From: Colin Ian King <colin.king at canonical.com>
>
> Minor change to fix cppcheck style warnings, no functional change
>
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
>  src/acpica/source/compiler/fwts_iasl_interface.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/src/acpica/source/compiler/fwts_iasl_interface.c b/src/acpica/source/compiler/fwts_iasl_interface.c
> index d0ffcb7..3be4146 100644
> --- a/src/acpica/source/compiler/fwts_iasl_interface.c
> +++ b/src/acpica/source/compiler/fwts_iasl_interface.c
> @@ -78,7 +78,7 @@ int fwts_iasl_disassemble_aml(
>  	const char *outputfile)
>  {
>  	pid_t	pid;
> -	int	status, i;
> +	int	status;
>  	FILE	*fpout, *fperr;
>
>  	fflush(stdout);
> @@ -105,6 +105,7 @@ int fwts_iasl_disassemble_aml(
>
>  		/* Do we need to include external tables in? */
>  		if (use_externals) {
> +			int i;
>  			/*
>  			 * Add in external SSDT files and NOT the one we want
>  			 * disassemble
>

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



More information about the fwts-devel mailing list