Re: add (void) cast inside advance_aggregates for function ExecEvalExprSwitchContext

2023-09-07 Thread Daniel Gustafsson
> On 3 Sep 2023, at 05:16, jian he wrote: > In src/backend/executor/nodeAgg.c > 817: advance_aggregates(AggState *aggstate) > > Do we need to add "(void)" before ExecEvalExprSwitchContext? I don't think we need to, but we could since we are in fact discardnig the return value. Did you get a

add (void) cast inside advance_aggregates for function ExecEvalExprSwitchContext

2023-09-02 Thread jian he
Hi. In src/backend/executor/nodeAgg.c 817: advance_aggregates(AggState *aggstate) Do we need to add "(void)" before ExecEvalExprSwitchContext?