Imath icon indicating copy to clipboard operation
Imath copied to clipboard

check return status of extractSHRT() in testMatrix()

Open cary-ilm opened this issue 3 years ago • 0 comments

testMatrix() in testExtractSHRT.cpp fails to check the return status of extractSHRT(). If the input matrix is invalid, it will produce garbage results. A better solution is to assert that extractSHRT() returns true.

Adding this assert will eliminate an uninitialized variable reference warning, since the s, h, r, and t variables may be unset.

cary-ilm avatar Mar 16 '22 01:03 cary-ilm