Patchwork acpica: fwts_acpica: use size_t instead of int

login
register
mail settings
Submitter Colin King
Date April 10, 2012, 2:34 p.m.
Message ID <1334068442-2576-1-git-send-email-colin.king@canonical.com>
Download mbox | patch
Permalink /patch/151600/
State Accepted
Headers show

Comments

Colin King - April 10, 2012, 2:34 p.m.
From: Colin Ian King <colin.king@canonical.com>

Signed-off-by: Colin Ian King <colin.king@canonical.com>
---
 src/acpica/fwts_acpica.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)
Alex Hung - April 11, 2012, 2:46 a.m.
On 04/10/2012 10:34 PM, Colin King wrote:
> From: Colin Ian King<colin.king@canonical.com>
>
> Signed-off-by: Colin Ian King<colin.king@canonical.com>
> ---
>   src/acpica/fwts_acpica.c |    4 ++--
>   1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/acpica/fwts_acpica.c b/src/acpica/fwts_acpica.c
> index 609dc8f..1e2db3c 100644
> --- a/src/acpica/fwts_acpica.c
> +++ b/src/acpica/fwts_acpica.c
> @@ -266,10 +266,10 @@ void fwts_acpica_debug_command(fwts_framework *fw, fwts_acpica_log_callback func
>   void fwts_acpica_vprintf(const char *fmt, va_list args)
>   {
>   	static char *buffer;
> -	static int  buffer_len;
> +	static size_t buffer_len;
>
>   	char tmp[4096];
> -	int tmp_len;
> +	size_t tmp_len;
>
>   	vsnprintf(tmp, sizeof(tmp), fmt, args);
>   	tmp_len = strlen(tmp);
Acked-by: Alex Hung <alex.hung@canonical.com>
Keng-Yu Lin - April 11, 2012, 5:14 a.m.
On Tue, Apr 10, 2012 at 10:34 PM, Colin King <colin.king@canonical.com> wrote:
> From: Colin Ian King <colin.king@canonical.com>
>
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
> ---
>  src/acpica/fwts_acpica.c |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/acpica/fwts_acpica.c b/src/acpica/fwts_acpica.c
> index 609dc8f..1e2db3c 100644
> --- a/src/acpica/fwts_acpica.c
> +++ b/src/acpica/fwts_acpica.c
> @@ -266,10 +266,10 @@ void fwts_acpica_debug_command(fwts_framework *fw, fwts_acpica_log_callback func
>  void fwts_acpica_vprintf(const char *fmt, va_list args)
>  {
>        static char *buffer;
> -       static int  buffer_len;
> +       static size_t buffer_len;
>
>        char tmp[4096];
> -       int tmp_len;
> +       size_t tmp_len;
>
>        vsnprintf(tmp, sizeof(tmp), fmt, args);
>        tmp_len = strlen(tmp);
> --
> 1.7.9.1
>
Acked-by: Keng-Yu Lin <kengyu@canonical.com>

Patch

diff --git a/src/acpica/fwts_acpica.c b/src/acpica/fwts_acpica.c
index 609dc8f..1e2db3c 100644
--- a/src/acpica/fwts_acpica.c
+++ b/src/acpica/fwts_acpica.c
@@ -266,10 +266,10 @@  void fwts_acpica_debug_command(fwts_framework *fw, fwts_acpica_log_callback func
 void fwts_acpica_vprintf(const char *fmt, va_list args)
 {
 	static char *buffer;
-	static int  buffer_len;
+	static size_t buffer_len;
 
 	char tmp[4096];
-	int tmp_len;
+	size_t tmp_len;
 
 	vsnprintf(tmp, sizeof(tmp), fmt, args);
 	tmp_len = strlen(tmp);