<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Wed, Jul 8, 2015 at 2:39 PM Krzysztof Parzyszek <<a href="mailto:kparzysz@codeaurora.org">kparzysz@codeaurora.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 6/24/2015 4:52 PM, Robinson, Paul wrote:<br>
><br>
> The patch causes all in-class-defined methods to be treated as if<br>
> they had the 'inline' keyword attached.<br>
> Therefore, with the patch, explicitly adding the 'inline' keyword to<br>
> these methods has no effect; it becomes noise.<br>
<br>
It is redundant.  Why is this a concern?  IIRC, the C++ standard has<br>
always defined in-class functions as "inline".<br>
I have always treated in-class functions as inline whenever I wrote<br>
them.  I'm surprised to hear that there are people who are unaware of<br>
this relationship, as this is a fairly basic feature of C++.<br>
I think we should treat these functions the way the standard requires it.<br></blockquote><div><br></div><div>Note, we already give these functions all of the semantics *required* by the standard for functionality. The *only* issue here is an optimization hint.</div></div></div>