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 v4 00/11] [PR gdb/14441] Support C++0x rvalue references in gdb


On 04/02/2016 09:47 AM, Artemiy Volkov wrote:
> On Thu, Mar 31, 2016 at 01:35:03PM -0700, Keith Seitz wrote:
>> On 03/21/2016 01:59 PM, Artemiy Volkov wrote:
>>> Hi all,
>>>
>>> this is my fourth take on fixing gdb/14441 which deals with C++0x rvalue
>>> references.
>>>
>>
>> There are two small nits (see replies in #8 and #9), otherwise, this
>> series looks ready for a final evaluation by a maintainer.
>>
>> Since the nits are really minor, I recommend just replying [to my
>> messages] with corrections on those two specific patches instead of
>> reposting the entire series.
> 
> Done and done. I hope that's OK that I didn't resend the description and
> changelog parts.
> 
>>
>> Thank you for your patch (and patience)!
> 
> Same here about the patience part :)

Thanks you very much for this patch set, and big thanks to Keith
for the review.

I read the series and it looks pretty good to me.  The only
thing I'm not sure about is the hardcoding of "-std=c++11"
in tests.  GCC 6 switches to -std=gnu++14  by default, for instance,
and I wouldn't want the tests to not cover whatever the compiler
uses by default.  Note also that that's -std=gnu++14, not the stricter
-std=c++14 mode.  Could we make the testcases gracefully skip rvalue
reference bits using #if __cplusplus?

BTW, please fix up all comments and commit logs to mention C++11
instead of C++0x.  :-)

Thanks,
Pedro Alves


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