commit 469bf392ad73e38dd23c03867269ede220e69d4a
author Sameer Agarwal <sameeragarwal@google.com> Wed Jun 13 22:21:23 2012 -0700
committer Sameer Agarwal <sameeragarwal@google.com> Thu Jun 14 06:59:46 2012 -0700
tree 44dbb12f015199f29a2d4a1bf85ea24728163c7e
parent 4441b5b2b823293da450160cbd2053fff64b4c66

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