Peter Eisentraut <pe...@eisentraut.org> writes: > If you change the member to a pointer
> - ErrorSaveContext escontext;
> + ErrorSaveContext *escontext;
> } JsonExprState;
> and make the required adjustments elsewhere in the code, the warning
> goes away.
> This arrangement would also appear to be more consistent with other
> executor nodes (e.g., ExprState, ExprEvalStep), so it might be worth it
> for consistency in any case.
+1, makes sense to me.
regards, tom lane
