On Mon, 24 Aug 2026 at 05:22, Michael Paquier <[email protected]> wrote:
>
> On Sun, Aug 23, 2026 at 06:38:58AM +0900, Michael Paquier wrote:
> >>  * nodes/extensible.h now includes nodes/readfuncs.h
> >>     I revised my opinion on this: I noticed my compiler complained
> >> about the plain "struct ReadNodeContext *ctx" argument if I didn't put
> >> a "struct ReadNodeContext;" declaration at the top level of the file,
> >> which (when combined with relevant comments) would add more verbose
> >> clutter than the alternative of including nodes/readfuncs.h.
>
> 89d091f602fa and 9673a0aa92f4 are now applied, for both changes.
>
> Thanks for this work, Matthias.

Thanks for all the feedback and committing, Michael!


Kind regards,

Matthias van de Meent
Databricks (https://www.databricks.com)


Reply via email to