Andres Freund <andres@anarazel.de> writes:
> Can't quite decide between just using always_inline - after all the buffer
> locking really is a rather crucial code path - and just initializing
> wait_event to 0. Either seems better than using a default:.
Yeah, I agree with not using a default: in case we ever extend
the mode enum. I'd be inclined to fix it like this:
case BUFFER_LOCK_UNLOCK:
pg_unreachable();
+ /* silence possible compiler warning: */
+ wait_event = 0;
}
regards, tom lane