Manfred Koizar <mkoi-pg(at)aon(dot)at> writes:
> Fix WriteBuffer() to return STATUS_OK/STATUS_ERROR instead of
> TRUE/FALSE. The return value is used by nextval() and do_setval()
> in sequence.c, all other callers ignore the return value.
Given the lack of any error checks in 99% of the callers, I do not think
this is an appropriate change. I'd vote for changing WriteBuffer to
return void, and have it elog() on bad argument. No one should ever
pass it a bogus buffer ID anyway --- if you don't have a valid buffer
ID, then what the heck were you just scribbling on?
regards, tom lane