Boost logo

Ublas :

From: Georg Baum (Georg.Baum_at_[hidden])
Date: 2007-09-21 03:58:55


Hi,

I just noticed (through this compiler warning about an unused variable:

/xxx/boost/numeric/ublas/matrix_sparse.hpp:2681: warning: unused parameter
‘preserve

) that there is at least one test for an unimplemented feature (in this case
preserve == true for compressed_matrix resizing) that is switched off in
release mode.

IMO such tests should always be done, because otherwise the user will get
wrong results without doing anything wrong. This kind of test is different
from all the other tests that test e.g. for out of range indices. Errors of
the latter kind are programming errors that do not need to be checked for
in release mode, but one must be prevented to use unimplemented features.

Georg