[cfe-commits] [cfe-dev] [review request] Removing redundant implicit casts in the AST, take 2
Nicola Gigante
nicola.gigante at gmail.com
Thu Apr 26 05:14:37 PDT 2012
Il giorno 25/apr/2012, alle ore 03:36, David Blaikie ha scritto:
> Bumping this because I think it's a neat thing to tidy up.
>
> I've attached a patch that's updated for r155507. The only really
> weird thing was a chunk of code in RewriteModernObjC.cpp that seemed
> like it was doing strictly the wrong thing for casts (& perhaps was
> only working as a no-op since explicit casts never did anything) &
> removing it fixed the break caused by this patch & didn't regress
> anything else.
>
> - David
Great! I stopped adapting the patch every new release because
no one seemed to care. Do you think it is ready to commit?
Anybody wants to comment?
Nicola
More information about the cfe-commits
mailing list