On 14.08.24 17:46, Paul Jungwirth wrote:
On 8/14/24 02:16, Peter Eisentraut wrote:
On 12.08.24 23:15, Paul Jungwirth wrote:
On 8/12/24 04:32, Aleksander Alekseev wrote:
[...] This function takes a Datum and the appropriate out function, and returns a char *. So you
can do this:

(gdb) call format_datum(range_out, $1)
$2 = 0x59162692d938 "[1,4)"

I assume a patch like this doesn't need documentation. Does it need a test? Anything else?

I think you forgot to attach the patch. Or is it just a proposal?

Sorry, patch attached here.

I don't think it's safe to call output functions at arbitrary points from a debugger.  But if you're okay with that during development, say, then I think you could just call OidOutputFunctionCall(F_RANGE_OUT, $1)?

I assumed it wasn't safe everywhere (e.g. there is potentially a TOAST lookup), but for debugging a patch that's okay with me.

Are you doing something to get macro expansion? I've never gotten my gdb to see #defines, although in theory this configure line should do it, right?:

Oh I see, you don't have the F_* constants available then. Maybe just put in the OID manually then?



Reply via email to