Patchwork [1/3] lib: fwts_framework: remove fwts_framework_setting

login
register
mail settings
Submitter Colin King
Date June 28, 2013, 12:04 a.m.
Message ID <1372377851-23390-2-git-send-email-colin.king@canonical.com>
Download mbox | patch
Permalink /patch/255254/
State Accepted
Headers show

Comments

Colin King - June 28, 2013, 12:04 a.m.
From: Colin Ian King <colin.king@canonical.com>

fwts_framework_setting is old and unused, so remove it

Signed-off-by: Colin Ian King <colin.king@canonical.com>
---
 src/lib/src/fwts_framework.c | 7 -------
 1 file changed, 7 deletions(-)
Keng-Yu Lin - June 28, 2013, 2:38 a.m.
On Fri, Jun 28, 2013 at 8:04 AM, Colin King <colin.king@canonical.com> wrote:
> From: Colin Ian King <colin.king@canonical.com>
>
> fwts_framework_setting is old and unused, so remove it
>
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
> ---
>  src/lib/src/fwts_framework.c | 7 -------
>  1 file changed, 7 deletions(-)
>
> diff --git a/src/lib/src/fwts_framework.c b/src/lib/src/fwts_framework.c
> index 7060c6b..add9eec 100644
> --- a/src/lib/src/fwts_framework.c
> +++ b/src/lib/src/fwts_framework.c
> @@ -88,13 +88,6 @@ static fwts_option fwts_framework_options[] = {
>
>  static fwts_list fwts_framework_test_list = FWTS_LIST_INIT;
>
> -typedef struct {
> -       const int env_id;
> -       const char *env_name;
> -       const char *env_default;
> -       char *env_value;
> -} fwts_framework_setting;
> -
>  static const char *fwts_copyright[] = {
>         "Some of this work - Copyright (c) 1999 - 2010, Intel Corp. All rights reserved.",
>         "Some of this work - Copyright (c) 2010 - 2013, Canonical.",
> --
> 1.8.3.1
>
Acked-by: Keng-Yu Lin <kengyu@canonical.com>
Alex Hung - June 28, 2013, 3:32 a.m.
On 06/28/2013 08:04 AM, Colin King wrote:
> From: Colin Ian King <colin.king@canonical.com>
> 
> fwts_framework_setting is old and unused, so remove it
> 
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
> ---
>  src/lib/src/fwts_framework.c | 7 -------
>  1 file changed, 7 deletions(-)
> 
> diff --git a/src/lib/src/fwts_framework.c b/src/lib/src/fwts_framework.c
> index 7060c6b..add9eec 100644
> --- a/src/lib/src/fwts_framework.c
> +++ b/src/lib/src/fwts_framework.c
> @@ -88,13 +88,6 @@ static fwts_option fwts_framework_options[] = {
>  
>  static fwts_list fwts_framework_test_list = FWTS_LIST_INIT;
>  
> -typedef struct {
> -	const int env_id;
> -	const char *env_name;
> -	const char *env_default;
> -	char *env_value;
> -} fwts_framework_setting;
> -
>  static const char *fwts_copyright[] = {
>  	"Some of this work - Copyright (c) 1999 - 2010, Intel Corp. All rights reserved.",
>  	"Some of this work - Copyright (c) 2010 - 2013, Canonical.",
> 
Acked-by: Alex Hung <alex.hung@canonical.com>

Patch

diff --git a/src/lib/src/fwts_framework.c b/src/lib/src/fwts_framework.c
index 7060c6b..add9eec 100644
--- a/src/lib/src/fwts_framework.c
+++ b/src/lib/src/fwts_framework.c
@@ -88,13 +88,6 @@  static fwts_option fwts_framework_options[] = {
 
 static fwts_list fwts_framework_test_list = FWTS_LIST_INIT;
 
-typedef struct {
-	const int env_id;
-	const char *env_name;
-	const char *env_default;
-	char *env_value;
-} fwts_framework_setting;
-
 static const char *fwts_copyright[] = {
 	"Some of this work - Copyright (c) 1999 - 2010, Intel Corp. All rights reserved.",
 	"Some of this work - Copyright (c) 2010 - 2013, Canonical.",