[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v5 04/14] tables.h: add linker table support



On Wed, Jan 04, 2017 at 11:47:14AM +0200, Andy Shevchenko wrote:
> On Tue, Jan 3, 2017 at 11:25 PM, Luis R. Rodriguez <mcgrof@xxxxxxxxxx> wrote:
> > On Thu, Dec 22, 2016 at 03:58:18PM +0200, Andy Shevchenko wrote:
> >> On Wed, 2016-12-21 at 18:38 -0800, Luis R. Rodriguez wrote:
> >> > +#define LINKTABLE_FOR_EACH(pointer, tbl)
> >>
> >> Hmm... SOMEONE LIKES CAPITAL LETTERS FOR everything, right? :-)
> >>
> >> I would expect more standard linktable_for_each() macro
> >
> > hpa had recommended this, if he prefers a lower case I can change that
> 
> 
> > but I really do consider this bikeshedding.
> 
> I don't think so:
> $ git grep -n FOR_EACH | wc -l
>     83
> $ git grep -n for_each | wc -l
>  23317

OK, would you also prefer if LINKTABLE_RUN_ERR() and LINKTABLE_RUN_ALL()
were lowercase ? Or is that fine ? To be clear I'll leave upper case the
declarers and defines.

  Luis

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.