Overlook in 2bd9e412?

Started by Julien Rouhaudabout 9 years ago2 messages
#1Julien Rouhaud
julien.rouhaud@dalibo.com
1 attachment(s)

It looks like pq_putmessage_hook and pq_flush_hook were meant for
development purpose and not supposed to be committed.

Attached patch remove them.

--
Julien Rouhaud
http://dalibo.com - http://dalibo.org

Attachments:

fix-extra-libpq-hooks.difftext/x-diff; name=fix-extra-libpq-hooks.diffDownload
diff --git a/src/include/libpq/libpq.h b/src/include/libpq/libpq.h
index 18052cb..5fac817 100644
--- a/src/include/libpq/libpq.h
+++ b/src/include/libpq/libpq.h
@@ -81,9 +81,6 @@ extern char *ssl_key_file;
 extern char *ssl_ca_file;
 extern char *ssl_crl_file;
 
-extern int	(*pq_putmessage_hook) (char msgtype, const char *s, size_t len);
-extern int	(*pq_flush_hook) (void);
-
 extern int	secure_initialize(void);
 extern bool secure_loaded_verify_locations(void);
 extern void secure_destroy(void);
#2Robert Haas
robertmhaas@gmail.com
In reply to: Julien Rouhaud (#1)
Re: Overlook in 2bd9e412?

On Fri, Oct 28, 2016 at 8:46 AM, Julien Rouhaud
<julien.rouhaud@dalibo.com> wrote:

It looks like pq_putmessage_hook and pq_flush_hook were meant for
development purpose and not supposed to be committed.

Attached patch remove them.

Thanks. Those were actually remnants of an earlier design which
didn't survive contact with Andres, and which I failed to clean up
properly.

Committed.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers