diff options
| author | Neil Conway <neilc@samurai.com> | 2004-11-14 02:04:14 +0000 |
|---|---|---|
| committer | Neil Conway <neilc@samurai.com> | 2004-11-14 02:04:14 +0000 |
| commit | a236dd9536bd4624b259759e08cb2b8d4585ab0f (patch) | |
| tree | 193c4b8d5962fb3c30fbc657c4650b67723d93b1 /src/backend/commands/sequence.c | |
| parent | 1c5125adf320316de46705a2f25548de2d20bbcc (diff) | |
| download | postgresql-a236dd9536bd4624b259759e08cb2b8d4585ab0f.tar.gz postgresql-a236dd9536bd4624b259759e08cb2b8d4585ab0f.zip | |
There is no need for ReadBuffer() call sites to check that the returned
buffer is valid, as ReadBuffer() will elog on error. Most of the call
sites of ReadBuffer() got this right, but this patch fixes those call
sites that did not.
Diffstat (limited to 'src/backend/commands/sequence.c')
| -rw-r--r-- | src/backend/commands/sequence.c | 9 |
1 files changed, 1 insertions, 8 deletions
diff --git a/src/backend/commands/sequence.c b/src/backend/commands/sequence.c index de71560d1da..98763cf0b2c 100644 --- a/src/backend/commands/sequence.c +++ b/src/backend/commands/sequence.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/commands/sequence.c,v 1.117 2004/09/16 16:58:28 tgl Exp $ + * $PostgreSQL: pgsql/src/backend/commands/sequence.c,v 1.118 2004/11/14 02:04:14 neilc Exp $ * *------------------------------------------------------------------------- */ @@ -192,10 +192,6 @@ DefineSequence(CreateSeqStmt *seq) /* Initialize first page of relation with special magic number */ buf = ReadBuffer(rel, P_NEW); - - if (!BufferIsValid(buf)) - elog(ERROR, "ReadBuffer failed"); - Assert(BufferGetBlockNumber(buf) == 0); page = (PageHeader) BufferGetPage(buf); @@ -850,9 +846,6 @@ read_info(SeqTable elm, Relation rel, Buffer *buf) Form_pg_sequence seq; *buf = ReadBuffer(rel, 0); - if (!BufferIsValid(*buf)) - elog(ERROR, "ReadBuffer failed"); - LockBuffer(*buf, BUFFER_LOCK_EXCLUSIVE); page = (PageHeader) BufferGetPage(*buf); |