Am 11.08.23 um 17:13 schrieb Jeff King:

On Thu, Aug 10, 2023 at 09:10:33PM +0200, René Scharfe wrote:

quoted

+#define DEFINE_PARSE_OPT_CB(name)				\
+static inline int name ## __void(const struct option *opt,	\
+				 const char *arg, int unset)	\
+{								\
+	return name(opt, arg, unset, opt->value);		\
+}								\
+struct option

BTW, I wondered what this extra "struct option" was doing. I guess it is
there to soak up the semi-colon of:

  DEFINE_PARSE_OPT_CB(foo);

with a noop declaration. Both clever and gross. :)

You guessed correctly.  It was short and handy..

A production-grade patch would use the function header.

René