On Thu, Jun 03, 2021 at 02:57:42PM +0200, Peter Eisentraut wrote:
> On 03.06.21 12:54, Bharath Rupireddy wrote:
> > It looks like for some of the fsm_set_and_search calls whose return
> > value is ignored (in fsm_search and RecordPageWithFreeSpace), there's
> > no (void). Is it intentional? In the code base, we generally have
> > (void) when non-void return value of a function is ignored.
>
> I don't think that is correct. I don't see anyone writing
>
> (void) printf(...);
We somehow do have a (void) fprint(...) in src/port/getopt.c.
> so this is not a generally applicable strategy.
>
> We have pg_nodiscard for functions where you explicitly want callers to
> check the return value. In all other cases, callers are free to ignore
> return values.
Yes, but we have a lot a examples of functions without pg_nodiscard and callers
still explicitly ignoring the results, like fsm_vacuum_page() in the same file.
It would be more consistent and make the code slightly more self explanatory.
