[cfe-commits] [PATCH] Fix some -Wunused-value false positives

Eli Friedman eli.friedman at gmail.com
Fri Oct 19 17:40:01 PDT 2012


On Fri, Oct 19, 2012 at 4:21 PM, Matt Beaumont-Gay <matthewbg at google.com> wrote:
> We were missing some cases in Expr::isUnusedResultAWarning, causing us
> to warn on the test case in the patch.

Looks fine.

> Any other Expr classes I should
> add to that switch, or any reason the current behavior is actually
> correct?

There probably are other classes you should add, but I don't know what
they are. :)

-Eli



More information about the cfe-commits mailing list