[PATCH v3 3/4] Introduce field_unsigned

Tom Tromey tromey@adacore.com
Wed Jul 17 13:31:00 GMT 2019


Pedro> Given the existence of "field_int", I'd expect "unsigned" to print an
Pedro> unsigned (int), not a ULONGEST.

Pedro> (I was initialize surprised it wasn't called field_uint, like the
Pedro> ui_out_field_uint function that is removed by this patch, until I
Pedro> realized this works with a ULONGEST.)

Pedro> To output a ULONGEST, I'd expect the method to be called field_ulongest.

Pedro> Is your plan to rename field_int to field_signed, perhaps?

I hadn't really considered it, but I can rename either one as you like.

I tend to think field_int should accept a LONGEST rather than an int.
So I suppose doing that & naming it field_signed would be a way to go.

What do think?

Tom



More information about the Gdb-patches mailing list