Thread: Could we do pgindent on just utils/adt/xml.c in the 8.3 branch?
I just committed a backpatch of http://archives.postgresql.org/pgsql-committers/2009-05/msg00229.php to 8.3. As in that patch, I did not try to manually reindent the code that's now wrapped in PG_TRY blocks. It might be a good idea to run pgindent on just that one file so that it's spacing-consistent with the later branches. This will ease any future backpatching in that file. regards, tom lane
Tom Lane wrote: > I just committed a backpatch of > http://archives.postgresql.org/pgsql-committers/2009-05/msg00229.php > to 8.3. As in that patch, I did not try to manually reindent the code > that's now wrapped in PG_TRY blocks. It might be a good idea to run > pgindent on just that one file so that it's spacing-consistent with > the later branches. This will ease any future backpatching in that > file. Sure, I can easily do that, but I am not sure how to generate a typedef list for that release, except to use the old typedef collection method for that file. Is that OK? -- Bruce Momjian <bruce@momjian.us> http://momjian.us EnterpriseDB http://enterprisedb.com PG East: http://www.enterprisedb.com/community/nav-pg-east-2010.do
Bruce Momjian <bruce@momjian.us> writes: > Sure, I can easily do that, but I am not sure how to generate a typedef > list for that release, except to use the old typedef collection method > for that file. Is that OK? One of the things I noticed while patching was that the 8.3 typedef list was missing xmlChar and a few other typedefs that are used a lot in that file :-(. So please don't use the same method you used in 8.3. I think it would be close enough to use the current (just added to CVS) typedef list. Or use the one you used for 8.4 if you have it handy. There is not a lot of difference between the 8.3 and current code in that file, so either of those are probably going to give you the same results. regards, tom lane
Tom Lane wrote: > Bruce Momjian <bruce@momjian.us> writes: > > Sure, I can easily do that, but I am not sure how to generate a typedef > > list for that release, except to use the old typedef collection method > > for that file. Is that OK? > > One of the things I noticed while patching was that the 8.3 typedef list > was missing xmlChar and a few other typedefs that are used a lot in that > file :-(. So please don't use the same method you used in 8.3. Yes, I don't have libxml installed on this machine so it didn't pick up those typedefs. > I think it would be close enough to use the current (just added to CVS) > typedef list. Or use the one you used for 8.4 if you have it handy. > There is not a lot of difference between the 8.3 and current code in > that file, so either of those are probably going to give you the same > results. Done. Sorry for the delay. -- Bruce Momjian <bruce@momjian.us> http://momjian.us EnterpriseDB http://enterprisedb.com PG East: http://www.enterprisedb.com/community/nav-pg-east-2010.do