tree 44dbb12f015199f29a2d4a1bf85ea24728163c7e
parent 4441b5b2b823293da450160cbd2053fff64b4c66
author Sameer Agarwal <sameeragarwal@google.com> 1339651283 -0700
committer Sameer Agarwal <sameeragarwal@google.com> 1339682386 -0700

Fixed an invalid DCHECK in suitesparse.cc

There was a overzealous DCHECK in suitesparse.cc when converting
a scalar matrix into a block matrix. This stemmed from my poor
understanding of how lower_bound works.

The test for this function was not stringent enough, and was
not run in debug mode for this to get triggered. The test
has been updated, it fails without the fix and runs correctly
with it.

Thanks to Markus Moll for reporting this and suggesting the
fix.

Change-Id: Ide6b971fd4c618ef5e240f500f514c4b78d7b6e3