Re: [HACKERS] ODBC drivers bundled with postgres

From: Bruce Momjian <maillist(at)candle(dot)pha(dot)pa(dot)us>
To: maillist(at)candle(dot)pha(dot)pa(dot)us (Bruce Momjian)
Cc: teunis(at)mauve(dot)computersupportcentre(dot)com, hackers(at)postgreSQL(dot)org
Subject: Re: [HACKERS] ODBC drivers bundled with postgres
Date: 1998-03-28 02:11:09
Message-ID: 199803280211.VAA26232@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

>
> >
> > These drivers are -very- corrupt in the image I downloaded... It looks
> > like someone ran a source reformatting program on it and completely
> > spammed it.
> >
> > As an example, all "//" comments that contained a "," were split into two
> > lines. All "//" comments containing the word "do" also got split.
> >
> > I could post patches - but I'd rather the maintainers for the package
> > handle it... I'm too likely to just reformat the entire beast.
> > [incidentally I finally got it to compile after 4+ hours of
> > manual patching]
>
> That was me. All *.c, *.h files are formatted via src/tools/pgindent.
> See the FAQ_DEV for more info. // are not valid C comments, as much as
> Windoze C programmers would like to think, and pgindent does not know
> how to handle them.
>
> C++ source is not reformatted. I was not aware pgindent actually was a
> problem for PostODBC. We can re-load the old sources and prevent
> pgindent from doing this in the future.

I am rolling back the pgindent changes, and will change the // comments
to /* */ comments, and run pgindent again.

--
Bruce Momjian | 830 Blythe Avenue
maillist(at)candle(dot)pha(dot)pa(dot)us | Drexel Hill, Pennsylvania 19026
+ If your life is a hard drive, | (610) 353-9879(w)
+ Christ can be your backup. | (610) 853-3000(h)

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Bruce Momjian 1998-03-28 02:13:09 pgindent on odbc
Previous Message Bruce Momjian 1998-03-28 01:57:37 Re: [HACKERS] ODBC drivers bundled with postgres