From eb6e2c11b16c9ef5287bda224fa965a37ef49131 Mon Sep 17 00:00:00 2001
From: Duane Robertson <duane@duanerobertson.com>
Date: Mon, 18 Jan 2016 01:55:08 -0600
Subject: [PATCH] Correct overflowing rivers in Valleys mapgen.

---
 src/mapgen_valleys.cpp | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/mapgen_valleys.cpp b/src/mapgen_valleys.cpp
index 93f47cd5f..6f4b2ccc7 100644
--- a/src/mapgen_valleys.cpp
+++ b/src/mapgen_valleys.cpp
@@ -670,8 +670,8 @@ int MapgenValleys::generateTerrain()
 
 	for (s16 z = node_min.Z; z <= node_max.Z; z++)
 	for (s16 x = node_min.X; x <= node_max.X; x++, index_2d++) {
-		s16 river_y = round(noise_rivers->result[index_2d]);
-		s16 surface_y = round(noise_terrain_height->result[index_2d]);
+		s16 river_y = floor(noise_rivers->result[index_2d]);
+		s16 surface_y = floor(noise_terrain_height->result[index_2d]);
 		float slope = noise_inter_valley_slope->result[index_2d];
 
 		heightmap[index_2d] = surface_y;
@@ -703,7 +703,7 @@ int MapgenValleys::generateTerrain()
 				} else if (river && y < river_y) {
 					// river
 					vm->m_data[index_data] = n_river_water;
-				} else if ((!fast_terrain) && (!river) && fill * slope > y - surface_y) {
+				} else if ((!fast_terrain) && (!river) && round(fill * slope) >= y - surface_y) {
 					// ground (slow method)
 					vm->m_data[index_data] = n_stone;
 					heightmap[index_2d] = surface_max_y = y;
-- 
GitLab