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: [RFA v2 16/17] Convert dwarf_expr_context_funcs to methods


>>>>> "Pedro" == Pedro Alves <palves@redhat.com> writes:

>> All I saw was the comment about possibly reducing churn:
>> https://sourceware.org/ml/gdb-patches/2016-10/msg00208.html

Pedro> I'm sorry, I had written comments about the parenthesis
Pedro> thing and also about wondering whether these methods with
Pedro> internal_error could be pure virtual ('= 0;') instead, and if not,
Pedro> about adding a small comment mentioning it.

I think the internal_error thing can't be a pure virtual because there
are subclasses that do not implement this method.  I could push the
internal error into the subclasses though.

Pedro> I sometimes write reviews of separate chunks in separate emails in
Pedro> then merge it all before sending.  Guess I deleted that email
Pedro> by mistake.

It's no problem.

Tom


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