Re: [PATCH v2 14/14] config: pass source to config_parser_event_fn_t

From: Jonathan Tan <jonathantanmy@google.com>
Date: Jun 2, 2023, 00:08 UTC
Message-ID: <20230602000841.434702-1-jonathantanmy@google.com>
In-Reply-To: <9e35b5b1f4dbdeafca5be7516477dc033b7643a6.1685472134.git.gitgitgadget@gmail.com>

"Glen Choo via GitGitGadget" <gitgitgadget@gmail.com> writes:

Show 6 quoted lines

> From: Glen Choo <chooglen@google.com>
> 
> ..so that the callback can use a "struct config_source" parameter
> instead of "config_reader.source". "struct config_source" is internal to
> config.c, but this refactor is okay because this function has only ever
> been (and probably ever will be) used internally by config.c.

Maybe s/this refactor/adding a pointer to a struct defined in a .c into a public function signature defined in a .h/

So this means that callers cannot instantiate that public function unless they are in config.c, but I see that it has been appropriately documented, so it should be fine.

Thanks for this series. Overall I had some minor comments, but things
look good overall.