Re: Review: more nan/inf checking (issue196066)
cku...@...
http://codereview.appspot.com/196066/diff/1/3
File src/liboslexec/exec.cpp (right): http://codereview.appspot.com/196066/diff/1/3#newcode345 src/liboslexec/exec.cpp:345: if (debugnan && check_nan (sym, NULL, 0, m_npoints-1, badval)) Can you pass the original run flags here? Otherwise you are checking points that might not be valid. http://codereview.appspot.com/196066/diff/1/3#newcode384 src/liboslexec/exec.cpp:384: if (!get_renderer_userdata (m_npoints, wants_derivatives, It would be good to check the results here too. http://codereview.appspot.com/196066/show |
|