On Tue, May 6, 2014 at 06:24:47PM -0400, Tom Lane wrote: > Bruce Momjian <br...@momjian.us> writes: > > On Tue, May 6, 2014 at 05:35:15PM -0400, Tom Lane wrote: > >> But I don't see the reason why we shouldn't expect ecpg's headers to > >> conform to our layout rules. > > > I don't know who ecpg got in there. Let me know what you would like > > done. > > What I'm suggesting is that, in HEAD only, you remove these exclusion > entries: > > > /ecpg/include/(sqlda|sqltypes)\.h$ > > /ecpg/include/preproc/struct\.h$ > > then redo the pgindent run (presumably only those three files will change) > and make any necessary updates in the ecpg expected files.
OK, done. ecpg exclusion removed, pgindent rerun (it only changed sqlda.h), and ecpg regression tests updated, all only in head. > Note that it's just chance that the back branch updates didn't hit those > three files already, since you said you weren't using the filter on them. Right. -- Bruce Momjian <br...@momjian.us> http://momjian.us EnterpriseDB http://enterprisedb.com + Everyone has their own god. + -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
