[Intel-gfx] [PATCH 1/3] lib/string_helpers: Consolidate yesno() implementation

Sakari Ailus sakari.ailus at linux.intel.com
Wed Jan 19 09:18:59 UTC 2022


Hi Lucas,

On Tue, Jan 18, 2022 at 11:24:48PM -0800, Lucas De Marchi wrote:
> @@ -1354,8 +1345,7 @@ static bool tomoyo_print_condition(struct tomoyo_io_buffer *head,
>  	case 3:
>  		if (cond->grant_log != TOMOYO_GRANTLOG_AUTO)
>  			tomoyo_io_printf(head, " grant_log=%s",
> -					 tomoyo_yesno(cond->grant_log ==
> -						      TOMOYO_GRANTLOG_YES));
> +					 yesno(cond->grant_log == TOMOYO_GRANTLOG_YES));

This would be better split on two lines.

Then,

Reviewed-by: Sakari Ailus <sakari.ailus at linux.intel.com>

-- 
Sakari Ailus


More information about the Intel-gfx mailing list