Re: ODBC Patch

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Dave Page <dpage(at)vale-housing(dot)co(dot)uk>
Cc: "'Tom Lane'" <tgl(at)sss(dot)pgh(dot)pa(dot)us>, "'pgsql-patches(at)postgresql(dot)org'" <pgsql-patches(at)postgresql(dot)org>
Subject: Re: ODBC Patch
Date: 2001-02-12 18:32:01
Message-ID: 200102121832.NAA08916@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches

> > > Can you send me that resource.h file again, and I will merge in any
> > > change made by *.rc, but keep the /* */ comments, OK?
> >
> > I don't think that's a feasible long-term maintenance approach either.
> > Will you or Dave remember to do this any time psqlodbc.rc changes?
>
> I can certainly do this anytime I update the driver, but I obviously can't
> keep track of other people who change it. IMHO it (psqlodbc.rc) should be
> updated for every patch submitted as it (along with psqlodbc.h) contain the
> revision number which is essential for the ODBC driver which is often
> updated outside of the main PostgreSQL releases.

It is my understanding that resource.h is changed only when a new
option is added t psqlodbc.rc. It does not change with a version number
change because resource.h doesn't contain a version number.

>
> > Seems like the correct answer is for pgindent to
> > automatically transform
> > "// comment" to "/* comment */".
>
> This sounds much more sensible.

Sure, I can do that. I can have pgindent convert:

// comment

to

/* comment */

Dave, can you send me the MS-generated resource.h for me to test with
and cvs commit? Thanks.

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

In response to

Browse pgsql-patches by date

  From Date Subject
Next Message Bruce Momjian 2001-02-12 18:37:39 Re: oid2name silly patch
Previous Message Bruce Momjian 2001-02-12 17:53:51 Re: ODBC Patch