This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [RFA 08/13] Use gdbpy_ref in python.c
On 11/20/2016 08:41 PM, Tom Tromey wrote:
> This changes more places in python.c to use gdbpy_ref.
>
> Additionally, previously gdbpy_apply_type_printers would return
> EXT_LANG_RC_ERROR if a type printer returned None. However, that
> doesn't seem correct to me; this patch changes it to return
> EXT_LANG_RC_NOP in this case.
Agreed, this is what the value printers do, AFAICT from
gdbpy_apply_val_pretty_printer.
Does that result in user/script-visible behavior? Should
this be covered by some test?
Anyway, LGTM.
Thanks,
Pedro Alves