Re: [eigen] Fix for broken asserts revealed by Clang

[ Thread Index | Date Index | More Archives ]

On Wed, 18 Jan 2012, Keir Mierle wrote:

The attached mercurial export fixes some broken asserts. It's surprising
that these ever worked as expected, since the asserts always evaluated to
true before. Clang forbids converting char pointers to bool, which is what
triggered this.

Well done. Turns out that I misunderstood the logic of the code when I added the assert, so the assert was actually triggered and silently ignored in some cases. Anyway, that's hopefully fixed now.


Mail converted by MHonArc 2.6.19+