The loop for randomizing a list in the dissolve transition is poor.
It randomly chooses two element and swaps them, and does this N/2 times.
Thus a total of N elements are moved.
It is probable that several elements will be swapped multiple times,
leaving it probable that several other elements will not be swapped at all.
This fact is quite visible when transitioning between two very different
slides (e.g. 2 photos). While there is a lot of randomness, there is a
clearly perceptible top-to-bottom 'wipe' happening at the same time.
So change the loop to provide true randomness.
We randomly choose an element for the last position.
Then randomly choose one of the remaining elements for the second
last position, etc.
Signed-off-by: NeilBrown <neilb@suse.de>
2006/10/18 19:59:01 thb 1.4.18.3: RESYNC: (1.4-1.5); FILE MERGED
2006/04/24 13:25:33 thb 1.4.18.2: #i53194# Unified include statements (local headers always have double quotes; external headers angle brackets); reverted EventMultiplexer pause events to shared_ptr; removed EventMultiplexer::removeViewHandler(), since the handler is held weakly, anyway.
2006/03/24 18:23:25 thb 1.4.18.1: #i37778# Moved whole slideshow engine from namespace presentation (which conflicts with one of the UNO subnamespaces) to slideshow
