ACK: [PATCH 4/5] lib: fwts_framework: constify categories[] and log_levels[]

Alex Hung alex.hung at canonical.com
Wed Oct 19 17:29:11 UTC 2016


On 2016-10-19 05:37 AM, Colin King wrote:
> From: Colin Ian King <colin.king at canonical.com>
>
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
>  src/lib/src/fwts_framework.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/lib/src/fwts_framework.c b/src/lib/src/fwts_framework.c
> index a296f93..992d901 100644
> --- a/src/lib/src/fwts_framework.c
> +++ b/src/lib/src/fwts_framework.c
> @@ -58,7 +58,7 @@ typedef struct {
>  	 FWTS_FLAG_TEST_ACPI |			\
>  	 FWTS_FLAG_TEST_COMPLIANCE_ACPI)
>
> -static fwts_categories categories[] = {
> +static const fwts_categories categories[] = {
>  	{ "ACPI",			FWTS_FLAG_TEST_ACPI },
>  	{ "Batch",			FWTS_FLAG_BATCH },
>  	{ "Interactive",		FWTS_FLAG_INTERACTIVE },
> @@ -72,7 +72,7 @@ static fwts_categories categories[] = {
>  	{ NULL,				0 },
>  };
>
> -static fwts_log_levels log_levels[] = {
> +static const fwts_log_levels log_levels[] = {
>  	{ "critical",			LOG_LEVEL_CRITICAL },
>  	{ "high",			LOG_LEVEL_CRITICAL | LOG_LEVEL_HIGH },
>  	{ "medium",			LOG_LEVEL_CRITICAL | LOG_LEVEL_HIGH | LOG_LEVEL_MEDIUM },
>


Acked-by: Alex Hung <alex.hung at canonical.com>



More information about the fwts-devel mailing list