[Lldb-commits] [PATCH] D55584: [LLDB] Simplify Boolean expressions
Jonas Devlieghere via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Wed Dec 12 09:22:23 PST 2018
JDevlieghere added a comment.
In D55584#1328087 <https://reviews.llvm.org/D55584#1328087>, @labath wrote:
> I also find the `static_cast<bool>` thingy weird. The rest of the changes seem to be towards the better (based on a pseudo-random sample), but the change is a quite big.
Me and Adrian went through all the changes and I didn't see anything that looked out of the ordinary.
I left the static cast because it was suggested to me before in a review, but happy to remove it as there is a consensus that it's worse :-)
CHANGES SINCE LAST ACTION
More information about the lldb-commits