The return value of SPI_connect - Mailing list pgsql-hackers

From Hou, Zhijie
Subject The return value of SPI_connect
Date
Msg-id 4bf7e10fcc0d4a7faf3143fde83fc577@G08CNEXMBPEKD05.g08.fujitsu.local
Whole thread Raw
Responses Re: The return value of SPI_connect  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
Hi

I found the function SPI_connect() has only one return value(SPI_OK_CONNECT).
But I also found that some places check the return value of SPI_connect() like the following:

    if (SPI_connect() != SPI_OK_CONNECT)
        elog(ERROR, "SPI_connect failed");

Is it necessary to check the return value of SPI_connect() ?

And in doc https://www.postgresql.org/docs/13/spi-spi-connect.html
It also says SPI_connect() can return SPI_ERROR_CONNECT on error.

May be we can make "int SPI_connect()" => "void SPI_connect()" and fix the doc ?

Best regards,
houzj





pgsql-hackers by date:

Previous
From: Abhijit Menon-Sen
Date:
Subject: [PATCH] SET search_path += octopus
Next
From: Amit Kapila
Date:
Subject: Re: New statistics for tuning WAL buffer size