* Baoquan He <b...@redhat.com> wrote:
> > > +static inline bool process_efi_entries(unsigned long minimum,
> > > + unsigned long image_size)
> >
> > ugly linebreak again ...
>
> The whole line is more than 80. I break the line and use tab and space
> to make it align with above 'unsigned long minimum'. Don't know why it
> becomes messy in patch. Will check and try again.
Then make the linebreak less ugly, or ignore the checkpatch warning!
This commonly used pattern:
static inline bool
process_efi_entries(unsigned long minimum, unsigned long image_size)
looks a lot better than the function parameter list broken in the middle.
Thanks,
Ingo
