diff options
| author | Simon Tatham <anakin@pobox.com> | 2016-01-03 09:51:15 +0000 |
|---|---|---|
| committer | Simon Tatham <anakin@pobox.com> | 2016-01-03 09:51:15 +0000 |
| commit | 55683abd97f94286982924ff0fecd03044e82bed (patch) | |
| tree | 9169c07477a6471b91382faf795e720adc8b4c63 /random.c | |
| parent | ff91d2d78d96441893acd1b1099726346e7f84cf (diff) | |
| download | puzzles-55683abd97f94286982924ff0fecd03044e82bed.zip puzzles-55683abd97f94286982924ff0fecd03044e82bed.tar.gz puzzles-55683abd97f94286982924ff0fecd03044e82bed.tar.bz2 puzzles-55683abd97f94286982924ff0fecd03044e82bed.tar.xz | |
Fix a valgrind warning in the Keen DIFF_HARD solver code.
The solver's array ctx->iscratch[] is used for a completely different
purpose in the DIFF_HARD code, compared to what it's used for in
DIFF_EASY and DIFF_NORMAL. In particular, a different number of
elements of the array are used - but the code which sets up the array
was not correctly initialising all of them.
I was also unable to find any clear comment that even explained _that_
the purpose of the array was entirely different between the two cases,
let alone explaining _what_ the two purposes were. So I've written
some comments as part of this commit, which should make things a bit
less confusing next time. (Though not much, I admit.)
Diffstat (limited to 'random.c')
0 files changed, 0 insertions, 0 deletions