On 01/17/2012 07:35 PM, Marti Raudsepp wrote:
> Here's my review for the "logging hooks" patch queued for the 2012-01
> CommitFest by Martin Pihlak.
>
Thanks for reviewing!
> There's a minor whitespace problem. When declaring variables, and the
> data type is longer than 12 characters, just use 1 single space
> character to delimit the variable name, instead of tab:
>
Fixed both in .h and .c
> I see that other hooks are declared with PGDLLIMPORT. I presume that
> this is necessary on Windows:
> extern PGDLLIMPORT emit_log_hook_type emit_log_hook;
Indeed, fixed now.
> I think the hook warrants a comment that, whether the messages will be
> seen, depends on the log_min_messages setting.
>
Comment added.
regards,
Martin