> 
> Yes, this is looking good. The patch didn't quite apply cleanly; if you can 
> bring it up to date with mainline I'll be happy to commit it, thanks!
> 
> Super-minor nit:
> 
> +    // CheckStaticCast _may_ have already created the cast node. Let's check
> +    if(CastNodesCreated) {
> 
> Please put a space after the "if" and before the "(" (in a bunch of places).
> 

Ok!
This should be the final patch. It applies to r145038 and pass the tests
(there is one that fails in any case in that revision, but it has nothing
to do with my patch).

>       - Doug

Thanks,
Nicola

Attachment: cast.patch
Description: Binary data

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

Reply via email to