[PATCH] D48318: [analyzer][UninitializedObjectChecker] Drop lambda support

2018-06-19 Thread Umann Kristóf via Phabricator via cfe-commits
Szelethus abandoned this revision. Szelethus added a comment. Turns out I was very wrong on this one. Lambdas should be ignored as data members, but not in general. Sorry about the spam, I think I should've sit longer on this one. I'll submit another patch tomorrow that will correctly explain m

[PATCH] D48318: [analyzer][UninitializedObjectChecker] Drop lambda support

2018-06-19 Thread Umann Kristóf via Phabricator via cfe-commits
Szelethus added a comment. > [...]lambda misuse in not really an uninitialized value problem. I guess you can make the argument that it its. Even then, in my opinion this checker is overkill for lambdas. - If the captured variable has a non-default constructor, `UninitializedValueChecker` will

[PATCH] D48318: [analyzer][UninitializedObjectChecker] Drop lambda support

2018-06-19 Thread Umann Kristóf via Phabricator via cfe-commits
Szelethus created this revision. Szelethus added reviewers: NoQ, george.karpenkov, xazax.hun, rnkovacs, dkrupp. Herald added subscribers: cfe-commits, mikhail.ramalho, a.sidorin, szepet, whisperity. After some thinking, I don't think this checker should support lambdas. Reason 1.: While this is