Messages in this thread | | | From | Bartosz Golaszewski <> | Date | Sat, 30 Jan 2021 21:25:08 +0100 | Subject | Re: [PATCH 5/8] lib: bitmap: remove the 'extern' keyword from function declarations |
| |
On Fri, Jan 29, 2021 at 4:59 PM Andy Shevchenko <andriy.shevchenko@linux.intel.com> wrote: > > On Fri, Jan 29, 2021 at 02:46:21PM +0100, Bartosz Golaszewski wrote: > > From: Bartosz Golaszewski <bgolaszewski@baylibre.com> > > > > The 'extern' keyword doesn't have any benefits in header files. Remove it. > > > +int __bitmap_equal(const unsigned long *bitmap1, > > + const unsigned long *bitmap2, unsigned int nbits); > > Why not > > int __bitmap_equal(const unsigned long *bitmap1, const unsigned long *bitmap2, > unsigned int nbits); > > and so on? > > It's even in 80 limit. >
I feel like this is purely a matter of taste. No rules define exactly how the lines should be broken. I prefer the longer part to be below, it just looks better to my eyes.
Bart
| |