diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2014-10-03 17:12:23 +0100 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2014-10-06 14:13:27 +0100 |
| commit | 8f436d3de7e99268a8862664d2cb2574231c3b18 (patch) | |
| tree | 0a07dd3cecbd7bc5b0293006bb8691e33c5cca75 /slideshow/source/inc | |
| parent | e5ab3685550cf35c3eb9cb47530044f2d86433d5 (diff) | |
use comphelper::rng::uniform_*_distribution everywhere
and automatically seed from time on first use
coverity#1242393 Don't call rand
coverity#1242404 Don't call rand
coverity#1242410 Don't call rand and additionally allow 0xFF as a value
coverity#1242409 Don't call rand
coverity#1242399 Don't call rand
coverity#1242372 Don't call rand
coverity#1242377 Don't call rand
coverity#1242378 Don't call rand
coverity#1242379 Don't call rand
coverity#1242382 Don't call rand
coverity#1242383 Don't call rand
coverity#1242402 Don't call rand
coverity#1242397 Don't call rand
coverity#1242390 Don't call rand
coverity#1242389 Don't call rand
coverity#1242388 Don't call rand
coverity#1242386 Don't call rand
coverity#1242384 Don't call rand
coverity#1242394 Don't call rand
Change-Id: I241feab9cb370e091fd6ccaba2af941eb95bc7cf
Diffstat (limited to 'slideshow/source/inc')
| -rw-r--r-- | slideshow/source/inc/tools.hxx | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/slideshow/source/inc/tools.hxx b/slideshow/source/inc/tools.hxx index 4e8037e3f3d3..682530f3bbc0 100644 --- a/slideshow/source/inc/tools.hxx +++ b/slideshow/source/inc/tools.hxx @@ -26,7 +26,7 @@ #include <com/sun/star/animations/XAnimationNode.hpp> #include <com/sun/star/container/XEnumerationAccess.hpp> #include <com/sun/star/container/XEnumeration.hpp> - +#include <comphelper/random.hxx> #include <vector> #include <cppcanvas/color.hxx> @@ -293,8 +293,7 @@ namespace slideshow /// Gets a random ordinal [0,n) inline ::std::size_t getRandomOrdinal( const ::std::size_t n ) { - return static_cast< ::std::size_t >( - double(n) * rand() / (RAND_MAX + 1.0) ); + return comphelper::rng::uniform_int_distribution(static_cast<size_t>(0), n-1); } /// To work around ternary operator in initializer lists |