This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [RFA 06/22] Introduce scoped_minimal_symbol_reader
- From: Tom Tromey <tom at tromey dot com>
- To: Pedro Alves <palves at redhat dot com>
- Cc: Tom Tromey <tom at tromey dot com>, gdb-patches at sourceware dot org
- Date: Mon, 10 Oct 2016 17:06:20 -0600
- Subject: Re: [RFA 06/22] Introduce scoped_minimal_symbol_reader
- Authentication-results: sourceware.org; auth=none
- References: <1474949330-4307-1-git-send-email-tom@tromey.com> <1474949330-4307-7-git-send-email-tom@tromey.com> <8343d689-83b9-646f-6503-d4d19a3d52ba@redhat.com>
>>>>> "Pedro" == Pedro Alves <palves@redhat.com> writes:
Pedro> GCC eschews using // comments, to avoid ending up with a mixbag
Pedro> of styles. Maybe we should follow suit?
I don't remember - did you want me to change this or not?
I prefer "//" comments but it's up to you.
>> +class scoped_minimal_symbol_reader
Pedro> Seems like this is growing to be _the_ reader in following patches?
Pedro> I'm wondering whether the "scoped_" prefix will make sense going
Pedro> forward.
I made this change. Thank goodness for git filter-branch.
Tom