From: | Michael Meskes <meskes(at)postgresql(dot)org> |
---|---|
To: | Takahiro Itagaki <itagaki(dot)takahiro(at)oss(dot)ntt(dot)co(dot)jp> |
Cc: | Boszormenyi Zoltan <zb(at)cybertec(dot)at>, Michael Meskes <meskes(at)postgresql(dot)org>, pgsql-hackers(at)postgresql(dot)org, Hans-Juergen Schoenig <hs(at)cybertec(dot)at> |
Subject: | Re: Fix auto-prepare #2 |
Date: | 2010-01-22 14:18:44 |
Message-ID: | 20100122141844.GA30579@feivel.credativ.lan |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Takahiro-san,
> Good. I think the patch is ready to commit.
Thanks for reviewing it. I just committed the patch.
> A comment for committer (Michael?) :
> I was cofused by the AddStmtToCache's 2nd argument "char *stmtID"
> because it doesn't have a const. Should it be "const char *" ?
> If the argument has a const, callers assume that they can pass
> a not-strdup'ed string as the argument.
Valid point, I can see no reason for not making this a "const char *". So let's
try.
Michael
--
Michael Meskes
Michael at Fam-Meskes dot De, Michael at Meskes dot (De|Com|Net|Org)
Michael at BorussiaFan dot De, Meskes at (Debian|Postgresql) dot Org
ICQ 179140304, AIM/Yahoo/Skype michaelmeskes, Jabber meskes(at)jabber(dot)org
VfL Borussia! Força Barça! Go SF 49ers! Use Debian GNU/Linux, PostgreSQL
From | Date | Subject | |
---|---|---|---|
Next Message | Greg Stark | 2010-01-22 14:19:47 | Re: About "Our CLUSTER implementation is pessimal" patch |
Previous Message | Alexey Klyukin | 2010-01-22 14:13:32 | Re: warn in plperl logs as... NOTICE?? |