In patch 0004, I noticed a couple of typos in the documentation; please find attached a fixup patch correcting these.

Still in the documentation, same patch, the last paragraph documenting PQcancelPoll() ends as:

+ indicate the current stage of the connection procedure and might be useful
+       to provide feedback to the user for example. These statuses are:
+      </para>

while not actually listing the "statuses". Should we list them? Adjust the wording? Or refer to PQconnectPoll() documentation (since the paragraph is copied from there it seems)?


Otherwise, the feature still works fine as far as I can tell.
From 3e04442e3f283829ed38e4a2b435fd182addf87a Mon Sep 17 00:00:00 2001
From: Denis Laxalde <denis.laxa...@dalibo.com>
Date: Wed, 6 Mar 2024 14:55:40 +0100
Subject: [PATCH] fixup! libpq: Add encrypted and non-blocking versions of
 PQcancel

---
 doc/src/sgml/libpq.sgml          | 2 +-
 src/interfaces/libpq/fe-cancel.c | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/doc/src/sgml/libpq.sgml b/doc/src/sgml/libpq.sgml
index 1613fcc7bb..1281cac284 100644
--- a/doc/src/sgml/libpq.sgml
+++ b/doc/src/sgml/libpq.sgml
@@ -6122,7 +6122,7 @@ PostgresPollingStatusType PQcancelPoll(PGcancelConn *cancelConn);
       <para>
        The request is made over the given <structname>PGcancelConn</structname>,
        which needs to be created with <xref linkend="libpq-PQcancelConn"/>.
-       The return value of <xref linkend="libpq-PQcancelBlocking"/>
+       The return value of <xref linkend="libpq-PQcancelStart"/>
        is 1 if the cancellation request could be started and 0 if not.
        If it was unsuccessful, the error message can be
        retrieved using <xref linkend="libpq-PQcancelErrorMessage"/>.
diff --git a/src/interfaces/libpq/fe-cancel.c b/src/interfaces/libpq/fe-cancel.c
index e66b8819ee..9c9a23bb4d 100644
--- a/src/interfaces/libpq/fe-cancel.c
+++ b/src/interfaces/libpq/fe-cancel.c
@@ -146,7 +146,7 @@ PQcancelBlocking(PGcancelConn *cancelConn)
 /*
  *		PQcancelStart
  *
- * Starts sending a cancellation request in a blocking fashion. Returns
+ * Starts sending a cancellation request in a non-blocking fashion. Returns
  * 1 if successful 0 if not.
  */
 int
-- 
2.39.2

Reply via email to