diff options
author | paramat <mat.gregory@virginmedia.com> | 2017-01-26 15:38:18 +0000 |
---|---|---|
committer | paramat <mat.gregory@virginmedia.com> | 2017-01-26 20:19:05 +0000 |
commit | 2a8953107181b4df6ff55d0ae214490575609f49 (patch) | |
tree | 269beb7799f54924b8bc5a84bbe040cd2694f7ee /src | |
parent | ae929ce2fdcfcfc95cffece35f3d38ea4d96dd9f (diff) | |
download | minetest-2a8953107181b4df6ff55d0ae214490575609f49.tar.gz minetest-2a8953107181b4df6ff55d0ae214490575609f49.tar.bz2 minetest-2a8953107181b4df6ff55d0ae214490575609f49.zip |
Dungeongen: Fix selection of diagonal corridors
The do .. while loop is waiting for both dir.X and dir.Z to be non-zero,
so should continue to loop if either dir.X or dir.Z are zero. The brackets
present suggest this was intended to be OR not AND.
Diffstat (limited to 'src')
-rw-r--r-- | src/dungeongen.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/dungeongen.cpp b/src/dungeongen.cpp index b8ee2b924..7825e9e2c 100644 --- a/src/dungeongen.cpp +++ b/src/dungeongen.cpp @@ -622,7 +622,7 @@ v3s16 rand_ortho_dir(PseudoRandom &random, bool diagonal_dirs) dir.Z = random.next() % 3 - 1; dir.Y = 0; dir.X = random.next() % 3 - 1; - } while ((dir.X == 0 && dir.Z == 0) && trycount < 10); + } while ((dir.X == 0 || dir.Z == 0) && trycount < 10); return dir; } else { |