Thread (8 messages) flat view 8 messages, 3 authors, 2021-10-06

Re: [PATCH iproute2 v3 1/3] configure: support --param=value style

From: Phil Sutter <phil@nwl.cc>
Date: 2021-10-06 10:19:07

Hi Andrea,

On Wed, Oct 06, 2021 at 11:49:34AM +0200, Andrea Claudi wrote:
[...]
That was my first proposal in v1 [1]. I changed it on David's suggestion
to consolidate the two cases into a single one.
Oh, sorry. I missed the 'v3' tag and hence didn't check any earlier
version.
Looking at the resulting code, v3 code results in an extra check to
discriminate between the two use cases, while v0 uses the "case"
structure to the same end.
Given that David explicitly requested the change I'm complaining about
in his reply to your initial version, I guess any further debate about
it is irrelevant.

Thanks, Phil
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help