Re: [RFA 22/22] Convert dwarf_expr_context_funcs to methods

Pedro Alves
2016-10-09T19:11:44Z

On 09/27/2016 05:08 AM, Tom Tromey wrote:
> This patch converts the function pointers in dwarf_expr_context_funcs
> into methods on dwarf_expr_context, and then updates the various
> implementations and callers to follow.
>
> NB this patch uses "override" (which caught a couple of renaming bugs
> during development) -- but this is C++11, so this patch at least has
> to wait for Pedro's patch that adds the OVERRIDE macro.

We can pull that bit in now, or remove the "override". No need to hold
this patch back.

LGTM. I wondered if defining member functions outside the
class definition wouldn't have resulted in a bit less churn
in dwarf2loc.c. gcc's coding conventions require doing that,
but I won't be pedantic about it.

Thanks,
Pedro Alves