[PATCH] Fixit for -Wreinterpret-case-class

Alexander Zinenko ftynse at gmail.com
Thu Mar 28 11:13:31 PDT 2013


Indeed, thanks!
Sorry that such a simple patch takes so much time.
Here is the updated one.

On 28 March 2013 19:53, Jordan Rose <jordan_rose at apple.com> wrote:

> Actually, I just realized that the SourceRange here is a token range,
> so CreateReplacement will do the right thing even if you just pass a single
> location. (I realized this because you shouldn't have to do the -1 offset;
> CharSourceRange is a half-open range like other C++ ranges.)
>
> Jordan
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130328/f8d1225a/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 13824fixit.patch
Type: application/octet-stream
Size: 10839 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130328/f8d1225a/attachment.obj>


More information about the cfe-commits mailing list