Hi All,

+1 for documenting it.

I took a look at the patch, it applies cleanly to master and the resulting
SGML is well-formed.

The description text mirrors pg_client_encoding()'s existing entry
("Returns current client encoding name."), which seems right to me for
consistency.  The declared "name" return type matches pg_proc.dat, too.

It might also be worth mentioning in the description that this returns the
server encoding, i.e. the same value as current_setting('server_encoding').
Feel free to ignore if you'd rather keep the entry minimal.

Regards,
Surya Poondla

Reply via email to