Tom Lane wrote:
> Alvaro Herrera <alvhe...@commandprompt.com> writes:
> > Andrew Dunstan escribi?:
> >> Here's the extract attached.  I replace tabs with a literal '\t' so
> >> I could see what it was doing. I can't make much head or tail of it
> >> either.
> 
> > pgindent uses entab/detab, which counts spaces and replaces them with
> > tabs.  It is wildly undocumented.  See src/tools/entab
> 
> What surprises me is that it seems to be changing lines that have been
> there for quite some time.  Bruce, have you been changing pgindent
> underneath us?

Not unless I am CVS committing without showing anyone.  ;-)

These are the only commits I see recently, which are harmless:

        revision 1.101
        date: 2009/06/11 22:21:44;  author: momjian;  state: Exp;  lines: +7 -2
        Document struct/union problem with pgindent.
        ----------------------------
        revision 1.100
        date: 2008/11/03 15:56:47;  author: momjian;  state: Exp;  lines: +2 -2
        Small shell syntax improvement.
        ----------------------------
        revision 1.99
        date: 2008/04/16 21:03:08;  author: momjian;  state: Exp;  lines: +2 -2
        Ignore blank lines in typedef file.
        ----------------------------
        revision 1.98
        date: 2008/01/16 20:13:44;  author: momjian;  state: Exp;  lines: +2 -2
        Improve usage message for pgindent.
        ----------------------------
        revision 1.97
        date: 2007/12/21 14:20:36;  author: momjian;  state: Exp;  lines: +11
        -2008
        Modify pgindent to use an external typedefs file rather than included
        list.

and entab is similarly unchanged:

        revision 1.18
        date: 2007/02/08 11:10:27;  author: petere;  state: Exp;  lines: +2 -2
        Normalize fgets() calls to use sizeof() for calculating the buffer size
        where possible, and fix some sites that apparently thought that fgets()
        will overwrite the buffer by one byte.
        
        Also add some strlcpy() to eliminate some weird memory handling.

I am stumped, but can keep researching.

-- 
  Bruce Momjian  <br...@momjian.us>        http://momjian.us
  EnterpriseDB                             http://enterprisedb.com

  + If your life is a hard drive, Christ can be your backup. +

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to