Re: [GENERAL] pgindented tsearch2 for 7.3.4 - Mailing list pgsql-hackers

From Bruce Momjian
Subject Re: [GENERAL] pgindented tsearch2 for 7.3.4
Date
Msg-id 200309241620.h8OGK5o13366@candle.pha.pa.us
Whole thread Raw
Responses Re: [GENERAL] pgindented tsearch2 for 7.3.4
Re: [GENERAL] pgindented tsearch2 for 7.3.4
List pgsql-hackers
Nigel J. Andrews wrote:
> 
> 
> I never knew running indent was so damn complicated. All three of my
> development systems can not manage it without throughing a fault (I've
> absolutely no idea why indent in the shell works but doesn't in pgindent on
> one).
> 
> Anyway, has anyone pgindented the version of tsearch2 that runs with 7.3.4? If
> so please email it to me. The tar file is tsearch-v2-stable.tar.gz, just for
> clarity and I can supply it if someone volunteers but doesn't have it.

There are about 6 files that can't be run through pgindent, and tsearch2
has one of them:
$ pgindent *.cHope you installed /src/tools/pgindent/indent.bsd.patch.gistidx.cError@186: Unbalanced parensWarning@187:
Extra)Error@314: Unbalanced parensWarning@315: Extra )Error@386: Unbalanced parensWarning@390: Extra )Error@427:
UnbalancedparensWarning@429: Extra )Error@439: Unbalanced parensWarning@440: Extra )Error@670: Unbalanced
parensWarning@671:Extra )Error@682: Unbalanced parensWarning@683: Extra )
 

It is actually this code that is causing it:
       LOOPBYTE(                if ((sign[i] & 0xff) != 0xff)                PG_RETURN_POINTER(retval);       );

With the macro being:
#define LOOPBYTE(a) \        for(i=0;i<SIGLEN;i++) {\                a;\        }

If you ask me, this is just _weird_ code and should not be allowed.

--  Bruce Momjian                        |  http://candle.pha.pa.us pgman@candle.pha.pa.us               |  (610)
359-1001+  If your life is a hard drive,     |  13 Roberts Road +  Christ can be your backup.        |  Newtown Square,
Pennsylvania19073
 


pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: invalid tid errors in latest 7.3.4 stable.
Next
From: Joe Conway
Date:
Subject: Re: creating/accessing new runtime parameters