Dmitri Gribenko <[email protected]> writes:

> On Thu, Jan 10, 2013 at 9:12 PM, David Greene <[email protected]> wrote:
>> Avoid a cast-away-const error by properly using const_cast<>.
>
> -  return forallBases(BaseIsNot, (void*) Base->getCanonicalDecl());
> +  return forallBases(BaseIsNot,
> +                     const_cast<CXXRecordDecl *>(Base->getCanonicalDecl()));
>
> LGTM, but a better solution would be to add an overload to
> forallBases() with const void * to hide this cast there.

I would like to handle that question separately if possible.  The code
owners should probably make these kinds of decisions.

                              -David
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to