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;
           }