[cfe-commits] Patch - Add fix-it hint for missing case keyword within a switch scope

[Richard Trieu]

Thanks for the comments, Doug.  I'll look into making the changes, just one
quick question.

There are two issues here. The first is that Expr::isIntegerConstantExpr()
> isn't safe for type- or value-dependent expressions, so we now crash on this
> ill-formed code:
>

Why does Clang crash here instead of returning false?  I didn't see a reason
listed in the source why it crashes or why Expr::isIntegerConstantExpr()
isn't safe for type- or value-dependent expressions.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20110420/4f45c70c/attachment.html>