ACK: [PATCH] lib: fwts_log_json: enable json human readable formatting (LP: #1206290)

[Alex Hung]

On 07/30/2013 06:31 AM, Colin King wrote:
> From: Colin Ian King <colin.king at canonical.com>
>
> The current json log output is hard to read as a human. Enable the json
> pretty printing to make it easier for mortals to parse by eye.
>
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
>   src/lib/src/fwts_log_json.c | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/src/lib/src/fwts_log_json.c b/src/lib/src/fwts_log_json.c
> index a67bc3a..15b28bd 100644
> --- a/src/lib/src/fwts_log_json.c
> +++ b/src/lib/src/fwts_log_json.c
> @@ -216,7 +216,8 @@ static void fwts_log_close_json(fwts_log_file *log_file)
>
>   	fwts_log_section_end_json(log_file);
>
> -	str = json_object_to_json_string(json_stack[0].obj);
> +	str = json_object_to_json_string_ext(
> +		json_stack[0].obj, JSON_C_TO_STRING_PRETTY);
>   	if (str == NULL) {
>   		/* Let's not make this bail out as user may be logging to other files too */
>   		fprintf(stderr, "Cannot turn json object to text for output. Empty json output\n");
>

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