r199169 - Consumed analysis: add two new attributes which fine-tune the behavior of

Delesley Hutchins delesley at google.com
Wed Jan 15 14:40:45 PST 2014


> Why is this code now commented out?

It's looked "fishy" to me ;-) and it didn't break any tests when I
removed it, but I left it there in case I ever discover why it was
there.

> I think you have to check whether the function is static or not as well.

Good point.  Thanks!

> These can be hoisted into the enclosing if.

Will do.

> Quite a few style guideline mishaps in here (extra newlines, curly
> braces when they should be elided, else clauses on their own line,
> etc). It might make sense to run clang-format over the file at some
> point.

Maybe, but we're not supposed to change SVN blame...

  -DeLesley

-- 
DeLesley Hutchins | Software Engineer | delesley at google.com | 505-206-0315



More information about the cfe-commits mailing list