Re: [Dbdpg-general] benchmarking old Pg and DBD::Pg - Mailing list pgsql-interfaces

Tom,

> > diff -ru DBD-Pg-1.41/types.c DBD-Pg-1.41-ed/types.c
> > --- DBD-Pg-1.41/types.c Fri Apr  8 16:06:23 2005
> > +++ DBD-Pg-1.41-ed/types.c Fri Apr  8 15:46:37 2005
> > @@ -55,7 +55,7 @@
> >         {ABSTIMEOID, "abstime", DBDPG_TRUE, null_quote, null_dequote, {0}},
> >         {RELTIMEOID, "reltime", DBDPG_TRUE, null_quote, null_dequote, {0}},
> >         {TINTERVALOID, "tinterval", DBDPG_TRUE, null_quote, null_dequote, {0}},
> > -       {UNKNOWNOID, "unknown", DBDPG_FALSE, null_quote, null_dequote, {0}},
> > +       {UNKNOWNOID, "unknown", DBDPG_FALSE, quote_varchar, null_dequote, {0}},
> >         {CIRCLEOID, "circle", DBDPG_FALSE, null_quote, null_dequote, {0}},
> >         {CASHOID, "money", DBDPG_TRUE, null_quote, null_dequote, {0}},
> >         {MACADDROID, "MAC address", DBDPG_TRUE,
> > quote_varchar,dequote_varchar, {0}},
> 
> Hmm ... if those columns mean what it looks like they mean, quite a few
> of the other entries seem wrong too.  All of the types visible in that
> fragment ought to be quoted when used as SQL literals.
> 
> Also, if the MACADDROID entry is any precedent, shouldn't you be
> changing the dequote field too?

I'm not with the development team of DBD::Pg to make those changes,
but I'm confident that it will be much appreciated by them (and myself
and other users of DBD::Pg for sure) if you could review its code in
regards to the type conversion issue and give your suggestions.

http://search.cpan.org/~dbdpg/DBD-Pg-1.41/


-- 
Vlad