On Tue, Oct 06, 2015 at 01:31:01PM -0000, Aaron Ballman via cfe-commits wrote:
> Log:
> Add a new module for the C++ Core Guidelines, and the first checker
> for those guidelines: cppcoreguidelines-pro-type-reinterpret-cast.

I wonder about the generality of this. Does it really make sense to flag
every reinterpret_cast of a void pointer? Such interface contracts
normally don't give you an alternative, so it seems a bit silly. This is
made worse by the concrete message used...

More importantly, can the checker detect all cases where
reinterpret_cast is used where a less generic cast applies?

Joerg
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to