From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | Mark Dilger <hornschnorter(at)gmail(dot)com> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: WIP patch for Oid formatting in printf/elog strings |
Date: | 2014-12-11 21:40:57 |
Message-ID: | 20141211214057.GI19832@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Thu, Dec 11, 2014 at 01:25:00PM -0800, Mark Dilger wrote:
>
>
> On Thu, Dec 11, 2014 at 12:12 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>
> Mark Dilger <hornschnorter(at)gmail(dot)com> writes:
> > The attached patch changes the type of chunk_seq to int32,
> > rather than changing the %d formatting. The other changes
> > are the same as in the previous patch.
>
> BTW, how are you finding these? If it's some automated tool, what?
> (If you're finding them by hand, I'm in awe of your scan rate.)
>
> regards, tom lane
>
>
> I've been advised to hoodwink you and say that I have found them all
> by hand. Actually, I am changing the typedef in src/include/c.h and then
> recompiling, and watching for compiler warnings telling me that I am
> passing a uint64 to a %d. Of course, I get lots of warnings about
> passing a uint64 to a %u, but I can filter through those easily enough.
Cool idea!
--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com
+ Everyone has their own god. +
From | Date | Subject | |
---|---|---|---|
Next Message | Alvaro Herrera | 2014-12-11 21:49:32 | Re: 9.5 release scheduling (was Re: logical column ordering) |
Previous Message | Mark Dilger | 2014-12-11 21:40:54 | Re: WIP patch for Oid formatting in printf/elog strings |