ACK: [PATCH 4/4] lib: klog: tidy up the code a little

[Alex Hung]

On 10/31/2012 10:11 PM, Colin King wrote:
> From: Colin Ian King <colin.king at canonical.com>
>
> The klog code needs a little bit of a tidy up, mostly using bools
> where possible and some re-formatting of the source - cosmetic
> changes.
>
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
>   src/lib/src/fwts_klog.c |   44 +++++++++++++++++++++++++-------------------
>   1 file changed, 25 insertions(+), 19 deletions(-)
>
> diff --git a/src/lib/src/fwts_klog.c b/src/lib/src/fwts_klog.c
> index 445ae4b..c611247 100644
> --- a/src/lib/src/fwts_klog.c
> +++ b/src/lib/src/fwts_klog.c
> @@ -20,6 +20,7 @@
>   #include <sys/klog.h>
>   #include <string.h>
>   #include <stdlib.h>
> +#include <stdbool.h>
>   #include <sys/types.h>
>   #include <pcre.h>
>   #include <json/json.h>
> @@ -35,7 +36,6 @@
>    */
>   #define KLOG_DATA_JSON_FILE		"klog.json"
>
> -
>   /*
>    *  fwts_klog_free()
>    *	free kernel log list
> @@ -100,23 +100,24 @@ char *fwts_klog_remove_timestamp(char *text)
>   }
>
>   int fwts_klog_scan(fwts_framework *fw,
> -		   fwts_list *klog,
> -		   fwts_klog_scan_func scan_func,
> -		   fwts_klog_progress_func progress_func,
> -		   void *private,
> -		   int *match)
> +	fwts_list *klog,
> +	fwts_klog_scan_func scan_func,
> +	fwts_klog_progress_func progress_func,
> +	void *private,
> +	int *match)
>   {
>   	typedef struct {
>   		char *line;
>   		int repeated;
>   	} klog_reduced_item;
>
> -	*match= 0;
>   	char *prev;
>   	fwts_list_link *item;
>   	fwts_list *klog_reduced;
>   	int i;
>
> +	*match = 0;
> +
>   	if (!klog)
>   		return FWTS_ERROR;
>
> @@ -132,7 +133,7 @@ int fwts_klog_scan(fwts_framework *fw,
>   		if (progress_func  && ((i % 25) == 0))
>   			progress_func(fw, 50 * i / fwts_list_len(klog));
>   		if (*newline) {
> -			int matched = 0;
> +			bool matched = false;
>   			fwts_list_link *l;
>   			fwts_list_foreach(l, klog_reduced) {
>   				char *line;
> @@ -141,7 +142,7 @@ int fwts_klog_scan(fwts_framework *fw,
>   				line = fwts_klog_remove_timestamp(reduced->line);
>   				if (strcmp(newline, line) == 0) {
>   					reduced->repeated++;
> -					matched = 1;
> +					matched = true;
>   					break;
>   				}
>   			}
> @@ -191,16 +192,19 @@ static char *fwts_klog_unique_label(const char *str)
>   	const char *src = str;
>   	char *dst;
>   	int count = 0;
> -	int forceupper = 1;
> +	bool forceupper = true;
>
>   	strcpy(buffer, "Klog");
>   	dst = buffer + 4;
>
> -	while ((dst < (buffer+sizeof(buffer)-1)) && (count < 4) && (*src)) {
> -		if ((*src == '|') || (*src == '/') || (*src == ' ')) {
> +	while ((dst < (buffer + sizeof(buffer) - 1)) &&
> +	       (count < 4) && (*src)) {
> +		if ((*src == '|') ||
> +		    (*src == '/') ||
> +		    (*src == ' ')) {
>   			src++;
>   			count++;
> -			forceupper = 1;
> +			forceupper = true;
>   			continue;
>   		}
>   		if (!isalnum(*src)) {
> @@ -210,7 +214,7 @@ static char *fwts_klog_unique_label(const char *str)
>   		*dst++ = forceupper ? toupper(*src) : *src;
>   		src++;
>
> -		forceupper = 0;
> +		forceupper = false;
>   	}
>   	*dst = '\0';
>   	return buffer;
> @@ -232,7 +236,7 @@ void fwts_klog_scan_patterns(fwts_framework *fw,
>   	FWTS_UNUSED(prevline);
>
>   	while (pattern->pattern != NULL) {
> -		int matched = 0;
> +		bool matched = false;
>   		switch (pattern->compare_mode) {
>   		case FWTS_COMPARE_REGEX:
>   			if (pattern->re)
> @@ -249,7 +253,9 @@ void fwts_klog_scan_patterns(fwts_framework *fw,
>   				fwts_log_info(fw, "Kernel message: %s", line);
>   			else {
>   				fwts_tag_failed(fw, pattern->tag);
> -				fwts_failed(fw, pattern->level, fwts_klog_unique_label(pattern->pattern), "%s Kernel message: %s", fwts_log_level_to_str(pattern->level), line);
> +				fwts_failed(fw, pattern->level,
> +					fwts_klog_unique_label(pattern->pattern),
> +					"%s Kernel message: %s", fwts_log_level_to_str(pattern->level), line);
>   				(*errors)++;
>   			}
>   			if (repeated)
> @@ -352,7 +358,7 @@ static int fwts_klog_check(fwts_framework *fw,
>   	}
>
>   	/* Now fetch json objects and compile regex */
> -	for (i=0; i<n; i++) {
> +	for (i = 0; i < n; i++) {
>   		const char *error;
>   		int erroffset;
>   		const char *str;
> @@ -386,7 +392,7 @@ static int fwts_klog_check(fwts_framework *fw,
>   		if (str) {
>   			patterns[i].label = strdup(str);
>   		} else {
> -			/* Not specified, so automagically generate */
> +			/* if not specified, auto-magically generate */
>   			patterns[i].label = strdup(fwts_klog_unique_label(patterns[i].pattern));
>   		}
>   		if (patterns[i].label == NULL)
> @@ -407,7 +413,7 @@ static int fwts_klog_check(fwts_framework *fw,
>   	ret = fwts_klog_scan(fw, klog, fwts_klog_scan_patterns, progress, patterns, errors);
>
>   fail:
> -	for (i=0; i<n; i++) {
> +	for (i = 0; i < n; i++) {
>   		if (patterns[i].re)
>   			pcre_free(patterns[i].re);
>   		if (patterns[i].extra)
>
Acked-by: Alex Hung <alex.hung at canonical.com>