Re: [HACKERS] pgindent bizarreness
Tom Lane wrote: Why has pgindent decided to screw up all the FD_SET calls in our code? See for example http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/postmaster/pgstat.c.diff?r1=1.188;r2=1.189 Because the typedef list supplied by Andrew includes FD_SET as a typedef. :-O See src/tools/pgindent/typedefs.list. -- Bruce Momjian br...@momjian.ushttp://momjian.us EnterpriseDB http://enterprisedb.com -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] pgindent bizarreness
Bruce Momjian wrote: Tom Lane wrote: Why has pgindent decided to screw up all the FD_SET calls in our code? See for example http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/postmaster/pgstat.c.diff?r1=1.188;r2=1.189 Because the typedef list supplied by Andrew includes FD_SET as a typedef. :-O See src/tools/pgindent/typedefs.list. I've committed a fix to pgindent for this. Do we want to rerun pgindent for these files?: src/test/examples/testlibpq2.c src/interfaces/libpq/fe-misc.c src/backend/postmaster/syslogger.c src/backend/postmaster/pgstat.c src/backend/postmaster/postmaster.c src/backend/replication/libpqwalreceiver/libpqwalreceiver.c src/backend/port/win32/socket.c src/backend/libpq/auth.c contrib/pgbench/pgbench.c cheers andrew -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] pgindent bizarreness
Andrew Dunstan and...@dunslane.net writes: I've committed a fix to pgindent for this. Do we want to rerun pgindent for these files?: I think the plan is to redo pgindent near the end of beta. There's probably no need to do it right now. regards, tom lane -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] pgindent bizarreness
Andrew Dunstan wrote: Bruce Momjian wrote: Tom Lane wrote: Why has pgindent decided to screw up all the FD_SET calls in our code? See for example http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/postmaster/pgstat.c.diff?r1=1.188;r2=1.189 Because the typedef list supplied by Andrew includes FD_SET as a typedef. :-O See src/tools/pgindent/typedefs.list. I've committed a fix to pgindent for this. Do we want to rerun pgindent for these files?: src/test/examples/testlibpq2.c src/interfaces/libpq/fe-misc.c src/backend/postmaster/syslogger.c src/backend/postmaster/pgstat.c src/backend/postmaster/postmaster.c src/backend/replication/libpqwalreceiver/libpqwalreceiver.c src/backend/port/win32/socket.c src/backend/libpq/auth.c contrib/pgbench/pgbench.c Uh, I rarely run pgindent on individual files like this. If it is FD_SET you are worried about, I suggest we find all the FD_SET symbols and just remove the whitespace pgindent added and commit that. -- Bruce Momjian br...@momjian.ushttp://momjian.us EnterpriseDB http://enterprisedb.com -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] pgindent bizarreness
Tom Lane wrote: Andrew Dunstan and...@dunslane.net writes: I've committed a fix to pgindent for this. Do we want to rerun pgindent for these files?: I think the plan is to redo pgindent near the end of beta. There's probably no need to do it right now. Sure, sounds like a plan. -- Bruce Momjian br...@momjian.ushttp://momjian.us EnterpriseDB http://enterprisedb.com -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] pgindent bizarreness
Tom Lane wrote: Why has pgindent decided to screw up all the FD_SET calls in our code? See for example http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/postmaster/pgstat.c.diff?r1=1.188;r2=1.189 This appears to be due to this on mingw: /mingw/include/winsock.h:typedef struct fd_set FD_SET; /mingw/include/winsock2.h:typedef struct fd_set FD_SET; We have a list of excluded symbols in the typdef finding code, which currently consists of this list: 'date','interval','timestamp','ANY' Looks like we should add 'FD_SET' to this list. I'm actually wondering if the list of excluded symbols should go in pgindent rather than the typedef finding code. It would be a very simple change to the pgindent script. cheers andrew -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers