On Tue, Dec 10, 2024 at 05:16:52PM +0100, Claudio Jeker wrote:
> On Mon, Dec 09, 2024 at 12:15:17PM +0100, Theo Buehler wrote:
> > > > > @@ -3536,6 +3539,7 @@ lookup(char *s)
> > > > >  		{ "export",		EXPORT},
> > > > >  		{ "export-target",	EXPORTTRGT},
> > > > >  		{ "ext-community",	EXTCOMMUNITY},
> > > > > +		{ "extended",		EXTENDED },
> > > > 
> > > > other members have no space before },
> > > 
> > > Actually that table is a mess. There are both versions. I want to unify
> > > this. I thought of adding a space before } for all of them. What do you
> > > prefer?
> > 
> > Right...
> > 
> > Removing the spaces in that one table is less churn, but adding one
> > seems more consistent with the rest of the file, where everything
> > seems to have a space (except addpathextra and addpathmax where there's
> > a tab).
> > 
> > While there, I'd also add a trailing comma to the tables that don't
> > have one: lookup[], toswords[], icmp_type[], icmp_code[]
> > 
> 
> How do you like the color of this bikeshed?

Am I allowed to say I like it, given that it's a bikeshed? I think it's
much better with the diff. Let's not waste more time on this

ok tb