On Thu, Dec 11, 2014 at 12:12 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
Mark Dilger <hornschnorter@gmail.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.
Also, the macros in outfuncs.c tend to complain in a similar way, and I can check that output, too.
