Patchwork lib: fwts_framework: use size_t instead of int

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

Comments

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

Signed-off-by: Colin Ian King <colin.king@canonical.com>
---
 src/lib/src/fwts_framework.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
Alex Hung - April 11, 2012, 2:14 a.m.
On 04/10/2012 10:43 PM, Colin King wrote:
> From: Colin Ian King<colin.king@canonical.com>
>
> Signed-off-by: Colin Ian King<colin.king@canonical.com>
> ---
>   src/lib/src/fwts_framework.c |    2 +-
>   1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/src/lib/src/fwts_framework.c b/src/lib/src/fwts_framework.c
> index e5be69d..0d0911b 100644
> --- a/src/lib/src/fwts_framework.c
> +++ b/src/lib/src/fwts_framework.c
> @@ -786,7 +786,7 @@ static void fwts_framework_heading_info(fwts_framework *fw, fwts_list *tests_to_
>   	time_t now;
>   	struct utsname buf;
>   	char *tests = NULL;
> -	int len = 1;
> +	size_t len = 1;
>   	int i;
>   	fwts_list_link *item;
>
Acked-by: Alex Hung <alex.hung@canonical.com>
Keng-Yu Lin - April 11, 2012, 5:10 a.m.
On Wed, Apr 11, 2012 at 10:14 AM, Alex Hung <alex.hung@canonical.com> wrote:
> On 04/10/2012 10:43 PM, Colin King wrote:
>>
>> From: Colin Ian King<colin.king@canonical.com>
>>
>> Signed-off-by: Colin Ian King<colin.king@canonical.com>
>> ---
>>  src/lib/src/fwts_framework.c |    2 +-
>>  1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/src/lib/src/fwts_framework.c b/src/lib/src/fwts_framework.c
>> index e5be69d..0d0911b 100644
>> --- a/src/lib/src/fwts_framework.c
>> +++ b/src/lib/src/fwts_framework.c
>> @@ -786,7 +786,7 @@ static void fwts_framework_heading_info(fwts_framework
>> *fw, fwts_list *tests_to_
>>        time_t now;
>>        struct utsname buf;
>>        char *tests = NULL;
>> -       int len = 1;
>> +       size_t len = 1;
>>        int i;
>>        fwts_list_link *item;
>>
> Acked-by: Alex Hung <alex.hung@canonical.com>
>
Acked-by: Keng-Yu Lin <kengyu@canonical.com>

Patch

diff --git a/src/lib/src/fwts_framework.c b/src/lib/src/fwts_framework.c
index e5be69d..0d0911b 100644
--- a/src/lib/src/fwts_framework.c
+++ b/src/lib/src/fwts_framework.c
@@ -786,7 +786,7 @@  static void fwts_framework_heading_info(fwts_framework *fw, fwts_list *tests_to_
 	time_t now;
 	struct utsname buf;
 	char *tests = NULL;
-	int len = 1;
+	size_t len = 1;
 	int i;
 	fwts_list_link *item;