Mercurial > octave
changeset 26348:d9e7affb27c8 stable
rand.cc: Fix static analyzer detected issues (bug #55347).
* rand.cc (do_rand): Don't use tmp.double_value () twice when intermediate
variable dval already contains this quantity. Don't call nint_big twice for
the same number.
author | Rik <rik@octave.org> |
---|---|
date | Tue, 01 Jan 2019 19:52:06 -0800 |
parents | 09a793a92c11 |
children | 6ec8b2eab987 |
files | libinterp/corefcn/rand.cc |
diffstat | 1 files changed, 1 insertions(+), 2 deletions(-) [+] |
line wrap: on
line diff
--- a/libinterp/corefcn/rand.cc Tue Jan 01 19:45:54 2019 -0800 +++ b/libinterp/corefcn/rand.cc Tue Jan 01 19:52:06 2019 -0800 @@ -177,8 +177,7 @@ dims.resize (2); - dims(0) = octave::math::nint_big (tmp.double_value ()); - dims(1) = octave::math::nint_big (tmp.double_value ()); + dims(0) = dims(1) = octave::math::nint_big (dval); goto gen_matrix; }