Andy Shevchenko
2022-Jan-19 19:25 UTC
[Nouveau] [PATCH 1/3] lib/string_helpers: Consolidate yesno() implementation
On Wed, Jan 19, 2022 at 10:06:35AM -0500, Steven Rostedt wrote:> On Wed, 19 Jan 2022 11:18:59 +0200 > Sakari Ailus <sakari.ailus at linux.intel.com> wrote: > > 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. > > Really? Yuck! > > I thought the "max line size" guideline was going to grow to a 100, but I > still see it as 80. But anyway... > > cond->grant_log => TOMOYO_GRANTLOG_YES > > is not readable at all. Not compared to > > cond->grant_log == TOMOYO_GRANTLOG_YES > > I say keep it one line! > > Reviewed-by: Steven Rostedt (Google) <rostedt at goodmis.org>I believe Sakari strongly follows the 80 rule, which means...> > Reviewed-by: Sakari Ailus <sakari.ailus at linux.intel.com>...chose either of these tags and be happy with :-) -- With Best Regards, Andy Shevchenko
Steven Rostedt
2022-Jan-19 21:00 UTC
[Nouveau] [PATCH 1/3] lib/string_helpers: Consolidate yesno() implementation
On Wed, 19 Jan 2022 21:25:08 +0200 Andy Shevchenko <andriy.shevchenko at linux.intel.com> wrote:> > I say keep it one line! > > > > Reviewed-by: Steven Rostedt (Google) <rostedt at goodmis.org> > > I believe Sakari strongly follows the 80 rule, which means...Checkpatch says "100" I think we need to simply update the docs (the documentation always lags the code ;-) bdc48fa11e46f -- Steve