[PATCH 1/2] arm-tdep: replace arm_mapping_symbol VEC with std::vector

Tom Tromey tom@tromey.com
Tue Jun 25 14:26:00 GMT 2019


>>>>> "Simon" == Simon Marchi <simon.marchi@polymtl.ca> writes:

Simon> This patch replaces VEC (arm_mapping_symbol) with an std::vector.  No
Simon> functional changes intended.

Thanks for doing this.  I had a couple of nits, nothing serious.

Simon>  struct arm_per_objfile
Simon>  {
Simon> -  VEC(arm_mapping_symbol_s) **section_maps;
Simon> +  arm_per_objfile (size_t num_sections)

Should be explicit.  Also probably this class should use
DISABLE_COPY_AND_ASSIGN.

thanks,
Tom



More information about the Gdb-patches mailing list