Andres Freund <and...@2ndquadrant.com> writes:
> On Tuesday, July 03, 2012 08:09:40 PM Tom Lane wrote:
>> If we really feel this is worth doing something about, we could invent a
>> formatting subroutine that converts XLogRecPtr to string (and then we
>> just use %s in the messages).

> I think that would make memory management annoying. Using a static buffer 
> isn't going to work very well either because its valid to pass two recptr's 
> to 
> elog/ereport/....

Hm.  I was assuming that we could probably get away with the
static-buffer trick, but if you think not ...

One possibility is to make call sites that need this pass local-variable
buffers to the formatting subroutine:

        char    xrp_buffer[XLOGRECPTR_BUF_LEN];
        char    xrp_buffer2[XLOGRECPTR_BUF_LEN];

        ereport(....,
                format_xlogrecptr(xrp_buffer, xlogval1),
                format_xlogrecptr(xrp_buffer2, xlogval2));

but it may not be worth the trouble.

                        regards, tom lane

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to