[cfe-commits] r151775 - /cfe/trunk/lib/StaticAnalyzer/Core/BugReporter.cpp

[David Blaikie]

Ah, right. Sorry I didn't see it was a fully covered switch - neat.
------------------------------
From: Ted Kremenek
Sent: 3/1/2012 11:42 AM
To: David Blaikie
Cc: cfe-commits at cs.uiuc.edu
Subject: Re: [cfe-commits] r151775 -
/cfe/trunk/lib/StaticAnalyzer/Core/BugReporter.cpp

On Feb 29, 2012, at 4:18 PM, David Blaikie <dblaikie at gmail.com> wrote:

For what it's worth - given the way if's boolean check on
initialization works, I kind of like the original expression (& hey,
it hapens to be shorter). Is there a particular reason for the
refactor? I wouldn't've thought it was faster...


I like the compiler telling me when I haven't covered all the cases.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120301/816633ba/attachment.html>