<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Nov 12, 2015 at 6:36 AM, Aaron Ballman via cfe-commits <span dir="ltr"><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">aaron.ballman added inline comments.<br>
<span class=""><br>
================<br>
Comment at: lib/AST/ASTContext.cpp:7930<br>
@@ -7931,3 +7929,3 @@<br>
<br>
-ASTMutationListener::~ASTMutationListener() { }<br>
+ASTMutationListener::~ASTMutationListener() = default;<br>
<br>
----------------<br>
</span><span class="">Eugene.Zelenko wrote:<br>
> aaron.ballman wrote:<br>
> > This is... interesting. Explicitly defaulting an out-of-line function definition is not something I've ever encountered before. What benefit does this provide?<br>
> It's explicitly tells that destructor has default implementation.<br>
</span>I'm not certain this is an improvement. Using =default in the declaration is an improvement because it tells the compiler up front "this has the default implementation" and the compiler can benefit from that information in other ways. When it's on an out-of-line definition, it does say "this has the default implementation", but it doesn't give the compiler any benefit over {}, so the only benefit is up to the reader of the code. I think someone can read {} to easily tell it is the default behavior, it is considerably shorter, and it doesn't cause anyone's eyes to trip over it wondering about the construct.<br></blockquote><div><br></div><div>If we're talking coding style, I'll vote marginally in favor of "= default" even in out of line definitions. But yeah, I don't think it necessarily adds much either *shrug* I probably wouldn't hold up a code review on it either way myself. (this is not a directive, just my own commentary)<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="im HOEnZb"><br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D14560" rel="noreferrer" target="_blank">http://reviews.llvm.org/D14560</a><br>
<br>
<br>
<br>
</span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
</div></div></blockquote></div><br></div></div>