>> Also the error is certainly user-facing,
>> so using elog() was quite inappropriate.  It should be ereport with an
>> errcode of (probably) ERRCODE_FEATURE_NOT_SUPPORTED.  Rolling your
>> own implementation of get_func_name() wasn't great either.
> 
> I overlooked the elog() call and "own implementation of
> get_func_name()". Will fix.

Attached is a trivial patch to fix that. I am going to push it if
there's no objection.

Best regards,
--
Tatsuo Ishii
SRA OSS K.K.
English: http://www.sraoss.co.jp/index_en/
Japanese:http://www.sraoss.co.jp

Attachment: v1-0001-Use-ereport-rather-than-elog-in-WinCheckAndInitia.patch
Description: Binary data

Reply via email to