> > Looks pretty good -- barring any objections I'll apply this to HEAD 
> > tomorrow or the day after (there's a room for a bit of 
> cleanup to the 
> > docs, but I'll do that before committing).
> 
> Couple of minor gripes:
> 
> 1.  Can we reduce the amount of stuff #include'd by pgstat.h? 
>  I particularly dislike including builtins.h there; that 
> surely should be unnecessary, since there's nothing but 
> externs in builtins.h.

Right. Only one needs to be there, the other two can be moved to
pgstatfuncs.c. The last one is needed to get the SockAddr structure.


> 2.  WRT the inet.h change: the project standard is that 
> extern declarations are spelled with "extern".

Fixed.

Updated patch attached.

//Magnus

Attachment: pgstat3.patch
Description: pgstat3.patch

---------------------------(end of broadcast)---------------------------
TIP 5: Have you checked our extensive FAQ?

               http://www.postgresql.org/docs/faq

Reply via email to