Andres Freund <[email protected]> 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
