This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH 6/8] Add support for the Rust language


On 04/27/2016 07:27 PM, Tom Tromey wrote:

>>> +static struct disr_info
>>> +rust_get_disr_info (struct type *type, const gdb_byte *valaddr,
> 
> Pedro> Did you mean to name these discr_info and rust_get_discr_info ?
> Pedro> I mean, the missing 'c'.
> 
> Manish wrote this part -- but not to blame him, this actually mirrors
> what is done in the Rust compiler.  I don't know why that abbreviation
> was chosen by Rust.  However, it is visible in the debuginfo, as enum
> discriminants can be named "RUST$ENUM$DISR".

Alright.  I'm fine with whatever.  It just looked odd to the
rust-untrained eye.

Thanks,
Pedro Alves


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]