Use "(void)" to mark pgstat_lock_entry(..., false) calls.
authorTom Lane <tgl@sss.pgh.pa.us>
Sun, 6 Apr 2025 15:37:09 +0000 (11:37 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Sun, 6 Apr 2025 15:37:09 +0000 (11:37 -0400)
commit2e4ccf1b4508cc337bb4d0afff1e32a049d549fc
tree531b8fbd5a7c806c7a49cc4c00f1c1dbdd8cd73c
parent5e1915439085014140314979c4dd5e23bd677cac
Use "(void)" to mark pgstat_lock_entry(..., false) calls.

This should silence Coverity's complaints about the result being
sometimes ignored.

I'm inclined to think that these routines are simply misdesigned,
because sometimes it's okay to ignore the result and sometimes it
isn't, and we have no way to enforce the latter.  But for now
I just added a comment.
src/backend/utils/activity/pgstat.c
src/backend/utils/activity/pgstat_database.c
src/backend/utils/activity/pgstat_shmem.c