<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Sep 1, 2014 at 10:29 PM, Justin Bogner <span dir="ltr"><<a href="mailto:mail@justinbogner.com" target="_blank">mail@justinbogner.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">Rafael Espíndola <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> writes:<br>

> It looks like there are code paths that are duplicated for<br>
> constructors and destructors just because<br>
> we have both CXXCtorType and CXXDtorsTypes.<br>
><br>
> This patch introduces a CDtorType enum that lets us reduce the<br>
> duplication a bit.<br>
<br>
</div></div>Looks like a pretty nice cleanup to me, but I find the CDtor naming a<br>
bit subtle. Not sure I have a much better idea, but a couple that come<br>
to mind are "COrDtor" or "CtorDtor". Alernatively we could go for<br>
something like CXXSpecial or CXXSpecMem, but these imply assignment as<br>
well, which is unfortunate.<br>
<br>
(feel free to veto me, I'm not convinced any of the names I've come up<br>
with are actually *better*)<br></blockquote><div><br></div><div>I had the same thought & think CtorDtorTypes is pretty reasonable.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</blockquote></div><br></div></div>