bug-fixin'

20 files changed, 336 insertions, 169 deletions

diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt
index 8325024ee..de8e93e04 100644
--- a/src/CMakeLists.txt
+++ b/src/CMakeLists.txt
@@ -30,7 +30,9 @@ else()
 	endif(BUILD_CLIENT)
 	find_package(ZLIB REQUIRED)
 	set(PLATFORM_LIBS -lpthread)
-	set(CLIENT_PLATFORM_LIBS -lXxf86vm)
+	#set(CLIENT_PLATFORM_LIBS -lXxf86vm)
+	find_library(XXF86VM_LIBRARY, Xxf86vm)
+	set(CLIENT_PLATFORM_LIBS ${XXF86VM_LIBRARY})
 endif()
 
 configure_file(
diff --git a/src/client.cpp b/src/client.cpp
index 915c745b7..4febdd4f7 100644
--- a/src/client.cpp
+++ b/src/client.cpp
@@ -279,10 +279,11 @@ void Client::step(float dtime)
 			// [0] u16 TOSERVER_INIT
 			// [2] u8 SER_FMT_VER_HIGHEST
 			// [3] u8[20] player_name
-			SharedBuffer<u8> data(2+1+20);
+			SharedBuffer<u8> data(2+1+PLAYERNAME_SIZE);
 			writeU16(&data[0], TOSERVER_INIT);
 			writeU8(&data[2], SER_FMT_VER_HIGHEST);
-			memcpy(&data[3], myplayer->getName(), 20);
+			memset((char*)&data[3], 0, PLAYERNAME_SIZE);
+			snprintf((char*)&data[3], PLAYERNAME_SIZE, "%s", myplayer->getName());
 			// Send as unreliable
 			Send(0, data, false);
 		}
diff --git a/src/defaultsettings.cpp b/src/defaultsettings.cpp
index 714703b08..e3ecf55f9 100644
--- a/src/defaultsettings.cpp
+++ b/src/defaultsettings.cpp
@@ -23,6 +23,9 @@ extern Settings g_settings;
 
 void set_default_settings()
 {
+	// Client and server
+	g_settings.setDefault("footprints", "false");
+
 	// Client stuff
 	g_settings.setDefault("wanted_fps", "30");
 	g_settings.setDefault("fps_max", "60");
@@ -37,7 +40,7 @@ void set_default_settings()
 	g_settings.setDefault("client_delete_unused_sectors_timeout", "1200");
 	g_settings.setDefault("enable_fog", "true");
 	g_settings.setDefault("new_style_water", "true");
-	g_settings.setDefault("terrain_viewer", "false");
+	g_settings.setDefault("new_style_leaves", "true");
 	
 	g_settings.setDefault("free_move", "false");
 	g_settings.setDefault("continuous_forward", "false");
diff --git a/src/environment.cpp b/src/environment.cpp
index b1722ee3f..e8a01a4a1 100644
--- a/src/environment.cpp
+++ b/src/environment.cpp
@@ -118,9 +118,9 @@ void Environment::step(float dtime)
 				/*
 					Apply water resistance
 				*/
-				if(player->in_water)
+				if(player->in_water_stable)
 				{
-					f32 max_down = 1.0*BS;
+					f32 max_down = 1.5*BS;
 					if(speed.Y < -max_down) speed.Y = -max_down;
 
 					f32 max = 2.5*BS;
@@ -155,27 +155,30 @@ void Environment::step(float dtime)
 			/*
 				Add footsteps to grass
 			*/
-			// Get node that is at BS/4 under player
-			v3s16 bottompos = floatToInt(playerpos + v3f(0,-BS/4,0));
-			try{
-				MapNode n = m_map->getNode(bottompos);
-				if(n.d == CONTENT_GRASS)
-				{
-					n.d = CONTENT_GRASS_FOOTSTEPS;
-					m_map->setNode(bottompos, n);
-#ifndef SERVER
-					// Update mesh on client
-					if(m_map->mapType() == MAPTYPE_CLIENT)
+			if(g_settings.getBool("footprints"))
+			{
+				// Get node that is at BS/4 under player
+				v3s16 bottompos = floatToInt(playerpos + v3f(0,-BS/4,0));
+				try{
+					MapNode n = m_map->getNode(bottompos);
+					if(n.d == CONTENT_GRASS)
 					{
-						v3s16 p_blocks = getNodeBlockPos(bottompos);
-						MapBlock *b = m_map->getBlockNoCreate(p_blocks);
-						b->updateMesh(m_daynight_ratio);
-					}
+						n.d = CONTENT_GRASS_FOOTSTEPS;
+						m_map->setNode(bottompos, n);
+#ifndef SERVER
+						// Update mesh on client
+						if(m_map->mapType() == MAPTYPE_CLIENT)
+						{
+							v3s16 p_blocks = getNodeBlockPos(bottompos);
+							MapBlock *b = m_map->getBlockNoCreate(p_blocks);
+							b->updateMesh(m_daynight_ratio);
+						}
 #endif
+					}
+				}
+				catch(InvalidPositionException &e)
+				{
 				}
-			}
-			catch(InvalidPositionException &e)
-			{
 			}
 		}
 	}
diff --git a/src/guiMainMenu.cpp b/src/guiMainMenu.cpp
index 2d02c0295..6c5d6553c 100644
--- a/src/guiMainMenu.cpp
+++ b/src/guiMainMenu.cpp
@@ -261,13 +261,13 @@ bool GUIMainMenu::OnEvent(const SEvent& event)
 			case 257: // Start game
 				acceptInput();
 				quitMenu();
-				break;
+				return true;
 			case 260: // Delete map
 				// Don't accept input data, just set deletion request
 				m_data->delete_map = true;
 				m_accepted = true;
 				quitMenu();
-				break;
+				return true;
 			}
 		}
 		if(event.GUIEvent.EventType==gui::EGET_EDITBOX_ENTER)
@@ -277,7 +277,7 @@ bool GUIMainMenu::OnEvent(const SEvent& event)
 			case 256: case 257: case 258:
 				acceptInput();
 				quitMenu();
-				break;
+				return true;
 			}
 		}
 	}
diff --git a/src/guiMessageMenu.cpp b/src/guiMessageMenu.cpp
index 7102ddd83..192911355 100644
--- a/src/guiMessageMenu.cpp
+++ b/src/guiMessageMenu.cpp
@@ -142,7 +142,7 @@ bool GUIMessageMenu::OnEvent(const SEvent& event)
 			case 257:
 				m_status = true;
 				quitMenu();
-				break;
+				return true;
 			}
 		}
 	}
diff --git a/src/guiPauseMenu.cpp b/src/guiPauseMenu.cpp
index bcf4070c9..486e4107d 100644
--- a/src/guiPauseMenu.cpp
+++ b/src/guiPauseMenu.cpp
@@ -193,15 +193,16 @@ bool GUIPauseMenu::OnEvent(const SEvent& event)
 			{

 			case 256: // continue

 				quitMenu();

-				break;

+				// ALWAYS return immediately after quitMenu()

+				return true;

 			case 260: // disconnect

 				m_gamecallback->disconnect();

 				quitMenu();

-				break;

+				return true;

 			case 257: // exit

 				m_gamecallback->exitToOS();

 				quitMenu();

-				break;

+				return true;

 			}

 		}

 	}

diff --git a/src/guiTextInputMenu.cpp b/src/guiTextInputMenu.cpp
index 38985cfa0..2cb8cae62 100644
--- a/src/guiTextInputMenu.cpp
+++ b/src/guiTextInputMenu.cpp
@@ -172,7 +172,8 @@ bool GUITextInputMenu::OnEvent(const SEvent& event)
 			case 257:
 				acceptInput();
 				quitMenu();
-				break;
+				// quitMenu deallocates menu
+				return true;
 			}
 		}
 		if(event.GUIEvent.EventType==gui::EGET_EDITBOX_ENTER)
@@ -182,7 +183,8 @@ bool GUITextInputMenu::OnEvent(const SEvent& event)
 			case 256:
 				acceptInput();
 				quitMenu();
-				break;
+				// quitMenu deallocates menu
+				return true;
 			}
 		}
 	}
diff --git a/src/main.cpp b/src/main.cpp
index 24f3664f6..17d481cac 100644
--- a/src/main.cpp
+++ b/src/main.cpp
@@ -112,19 +112,12 @@ Documentation:
 Build system / running:

 -----------------------

 

-NOTE: The following fixme is not apparently valid, and it does work.

-FIXME: Graphical mode seems to segfault with Irrlicht 1.7.1 on 64-bit

-       systems. (Ubuntu)

-       - http://pastebin.no/32bo

-	   - Might be just a bad build, too

-	   - Doesn't affect Irrlicht 1.7.2 or 32-bit 1.7.1. (Arch/Debian)

-	   - A similar error occurs when getTexture is called from a thread

-	     when the texture has not been already loaded from disk:

-		 http://irrlicht.sourceforge.net/phpBB2/viewtopic.php?p=68830

-

 FIXME: Some network errors on Windows that cause local game to not work

        - See siggjen's emails.

 	   - Is this the famous "windows 7 problem"?

+       - Apparently there might be other errors too

+	   - There is some problem with the menu system, something like the

+	     .Parent of guiPauseMenu to end up being 0xfeeefeee

 

 Networking and serialization:

 -----------------------------

@@ -254,26 +247,10 @@ TODO: Flowing water to actually contain flow direction information
 TODO: Remove duplicate lighting implementation from Map (leave

       VoxelManipulator, which is faster)

 

-FEATURE: Map generator version 2

-	- Create a system that allows a huge amount of different "map

-	  generator modules/filters"

-

-FEATURE: The map could be generated procedually:

-      - This would need the map to be generated in larger pieces

-	    - How large? How do they connect to each other?

-		- It has to be split vertically also

-		- Lighting would not have to be necessarily calculated until

-		  the blocks are actually needed - it would be quite fast

-		- Something like 64*64*16 MapBlocks?

-		  - No, MapSectors. And as much as it is efficient to do,

-		    64x64 might be too much.

-        - FIXME: This is currently halfway done and the generator is

-		  fairly broken

-      * Make the stone level with a heightmap

-	  * Carve out stuff in the stone

-	  * Dump dirt all around, and simulate it falling off steep

-	    places

-	  * Erosion simulation at map generation time

+FEATURE: Create a system that allows a huge amount of different "map

+	     generator modules/filters"

+

+FEATURE: Erosion simulation at map generation time

 		- Simulate water flows, which would carve out dirt fast and

 		  then turn stone into gravel and sand and relocate it.

 		- How about relocating minerals, too? Coal and gold in

@@ -294,8 +271,7 @@ Doing now (most important at the top):
   - map/meta.txt, which should contain only plain text, something like this:

       seed = 7ff1bafcd7118800

       chunksize = 8

-  - map/chunks.dat

-* Save chunk metadata on disk

+  - map/chunks.dat: chunk positions and flags in binary format

 * Make server find the spawning place from the real map data, not from

   the heightmap

   - But the changing borders of chunk have to be avoided, because

@@ -306,7 +282,6 @@ Doing now (most important at the top):
 * Check the fixmes in the list above

 

 === Stuff to do after release

-* Set backface culling on, especially for water

 * Add some kind of erosion and other stuff that now is possible

 * Make client to fetch stuff asynchronously

   - Needs method SyncProcessData

@@ -316,7 +291,7 @@ Doing now (most important at the top):
 * Water doesn't start flowing after map generation like it should

   - Are there still problems?

 * Better water generation (spread it to underwater caverns but don't

-  fill dungeons that don't touch outside air)

+  fill dungeons that don't touch big water masses)

 * When generating a chunk and the neighboring chunk doesn't have mud

   and stuff yet and the ground is fairly flat, the mud will flow to

   the other chunk making nasty straight walls when the other chunk

@@ -1943,7 +1918,9 @@ int main(int argc, char *argv[])
 		continue;

 	}

 	

-	std::cout<<DTIME<<"Connecting to server..."<<std::endl;

+	dstream<<DTIME<<"Connecting to server at ";

+	connect_address.print(&dstream);

+	dstream<<std::endl;

 	client.connect(connect_address);

 	

 	try{

diff --git a/src/map.cpp b/src/map.cpp
index ef6263c73..b51f1dac2 100644
--- a/src/map.cpp
+++ b/src/map.cpp
@@ -1455,8 +1455,8 @@ void Map::transformLiquids(core::map<v3s16, MapBlock*> & modified_blocks)
 	u32 loopcount = 0;
 	u32 initial_size = m_transforming_liquid.size();
 	
-	if(initial_size != 0)
-		dstream<<"transformLiquids(): initial_size="<<initial_size<<std::endl;
+	/*if(initial_size != 0)
+		dstream<<"transformLiquids(): initial_size="<<initial_size<<std::endl;*/
 
 	while(m_transforming_liquid.size() != 0)
 	{
@@ -1912,7 +1912,8 @@ void make_tree(VoxelManipulator &vmanip, v3s16 p0)
 	p1.Y -= 1;
 
 	VoxelArea leaves_a(v3s16(-2,-2,-2), v3s16(2,2,2));
-	SharedPtr<u8> leaves_d(new u8[leaves_a.getVolume()]);
+	//SharedPtr<u8> leaves_d(new u8[leaves_a.getVolume()]);
+	Buffer<u8> leaves_d(leaves_a.getVolume());
 	for(s32 i=0; i<leaves_a.getVolume(); i++)
 		leaves_d[i] = 0;
 	
@@ -1974,7 +1975,7 @@ double tree_amount_2d(u64 seed, v2s16 p)
 	double noise = noise2d_perlin(
 			0.5+(float)p.X/250, 0.5+(float)p.Y/250,
 			seed+2, 5, 0.6);
-	double zeroval = -0.4;
+	double zeroval = -0.3;
 	if(noise < zeroval)
 		return 0;
 	else
@@ -2009,11 +2010,17 @@ double base_rock_level_2d(u64 seed, v2s16 p)
 {
 	// The base ground level
 	double base = WATER_LEVEL - 4.0 + 25. * noise2d_perlin(
-			0.5+(float)p.X/500., 0.5+(float)p.Y/500.,
-			seed, 6, 0.6);
-
+			0.5+(float)p.X/250., 0.5+(float)p.Y/250.,
+			(seed>>32)+654879876, 6, 0.6);
+	/*// A bit hillier one
+	double base2 = WATER_LEVEL - 4.0 + 40. * noise2d_perlin(
+			0.5+(float)p.X/250., 0.5+(float)p.Y/250.,
+			(seed>>27)+90340, 6, 0.69);
+	if(base2 > base)
+		base = base2;*/
+#if 1
 	// Higher ground level
-	double higher = WATER_LEVEL + 23. + 30. * noise2d_perlin(
+	double higher = WATER_LEVEL + 13. + 50. * noise2d_perlin(
 			0.5+(float)p.X/500., 0.5+(float)p.Y/500.,
 			seed+85039, 6, 0.69);
 	//higher = 30; // For debugging
@@ -2021,27 +2028,36 @@ double base_rock_level_2d(u64 seed, v2s16 p)
 	// Limit higher to at least base
 	if(higher < base)
 		higher = base;
-	
+		
 	// Steepness factor of cliffs
 	double b = 1.0 + 1.0 * noise2d_perlin(
 			0.5+(float)p.X/250., 0.5+(float)p.Y/250.,
 			seed-932, 7, 0.7);
 	b = rangelim(b, 0.0, 1000.0);
-	// Make steep stuff very steep and non-steep stuff very non-steep
-	b = pow(b, 4);
-	b *= 10;
+	b = pow(b, 5);
+	b *= 7;
+	b = rangelim(b, 3.0, 1000.0);
+	//dstream<<"b="<<b<<std::endl;
 	//double b = 20;
 
+	// Offset to more low
+	double a_off = -0.3;
 	// High/low selector
-	double a = 0.5 + b * noise2d_perlin(
+	/*double a = 0.5 + b * (a_off + noise2d_perlin(
 			0.5+(float)p.X/500., 0.5+(float)p.Y/500.,
-			seed-359, 6, 0.7);
+			seed-359, 6, 0.7));*/
+	double a = 0.5 + b * (a_off + noise2d_perlin(
+			0.5+(float)p.X/250., 0.5+(float)p.Y/250.,
+			seed-359, 5, 0.60));
+	// Limit
 	a = rangelim(a, 0.0, 1.0);
 
 	//dstream<<"a="<<a<<std::endl;
 	
 	double h = base*(1.0-a) + higher*a;
-
+#else
+	double h = base;
+#endif
 	return h;
 }
 
@@ -2397,12 +2413,12 @@ MapChunk* ServerMap::generateChunkRaw(v2s16 chunkpos,
 	/*
 		Make dungeons
 	*/
-	u32 dungeons_count = relative_volume / 600000;
-	u32 bruises_count = relative_volume * stone_surface_max_y / 40000000;
+	//u32 dungeons_count = relative_volume / 600000;
+	/*u32 bruises_count = relative_volume * stone_surface_max_y / 40000000;
 	if(stone_surface_max_y < WATER_LEVEL)
-		bruises_count = 0;
-	//dungeons_count = 0;
-	//bruises_count = 0;
+		bruises_count = 0;*/
+	u32 dungeons_count = 0;
+	u32 bruises_count = 0;
 	for(u32 jj=0; jj<dungeons_count+bruises_count; jj++)
 	{
 		s16 min_tunnel_diameter = 2;
@@ -2536,7 +2552,7 @@ MapChunk* ServerMap::generateChunkRaw(v2s16 chunkpos,
 						//s16 si2 = rs - MYMAX(0, maxabsxz-rs/4);
 						s16 si2 = rs - MYMAX(0, maxabsxz-rs/7);
 						//s16 si2 = rs - abs(x0);
-						for(s16 y0=-si2+1; y0<=si2-1; y0++)
+						for(s16 y0=-si2+1+1; y0<=si2-1; y0++)
 						{
 							s16 z = cp.Z + z0;
 							s16 y = cp.Y + y0;
@@ -2723,7 +2739,7 @@ MapChunk* ServerMap::generateChunkRaw(v2s16 chunkpos,
 				m_seed+1, 3, 0.55));
 
 		// Find ground level
-		s16 surface_y = find_ground_level(vmanip, p2d);
+		s16 surface_y = find_ground_level_clever(vmanip, p2d);
 
 		/*
 			If topmost node is grass, change it to mud.
@@ -3031,7 +3047,7 @@ MapChunk* ServerMap::generateChunkRaw(v2s16 chunkpos,
 				0.5+(float)p2d.X/500, 0.5+(float)p2d.Y/500,
 				m_seed+59420, 3, 0.50);
 
-		bool have_sand = (sandnoise > 0.0);
+		bool have_sand = (sandnoise > -0.15);
 
 		if(have_sand == false)
 			continue;
diff --git a/src/mapblock.cpp b/src/mapblock.cpp
index fdc30dfd1..e3e90a9a1 100644
--- a/src/mapblock.cpp
+++ b/src/mapblock.cpp
@@ -210,10 +210,10 @@ void MapBlock::makeFastFace(TileSpec tile, u8 light, v3f p,
 	v3f vertex_pos[4];
 	// If looking towards z+, this is the face that is behind
 	// the center point, facing towards z+.
-	vertex_pos[0] = v3f( BS/2,-BS/2,BS/2);
-	vertex_pos[1] = v3f(-BS/2,-BS/2,BS/2);
-	vertex_pos[2] = v3f(-BS/2, BS/2,BS/2);
-	vertex_pos[3] = v3f( BS/2, BS/2,BS/2);
+	vertex_pos[0] = v3f(-BS/2,-BS/2,BS/2);
+	vertex_pos[1] = v3f( BS/2,-BS/2,BS/2);
+	vertex_pos[2] = v3f( BS/2, BS/2,BS/2);
+	vertex_pos[3] = v3f(-BS/2, BS/2,BS/2);
 	
 	if(dir == v3s16(0,0,1))
 	{
@@ -608,8 +608,13 @@ void MapBlock::updateMesh(u32 daynight_ratio)
 		JMutexAutoLock lock(m_temp_mods_mutex);
 		m_temp_mods.copy(temp_mods);
 	}
-
+	
+	/*
+		Some settings
+	*/
 	bool new_style_water = g_settings.getBool("new_style_water");
+	bool new_style_leaves = g_settings.getBool("new_style_leaves");
+	
 	float node_water_level = 1.0;
 	if(new_style_water)
 		node_water_level = 0.9;
@@ -695,6 +700,7 @@ void MapBlock::updateMesh(u32 daynight_ratio)
 		material.BackfaceCulling = false;
 		material.setFlag(video::EMF_BILINEAR_FILTER, false);
 		material.setFlag(video::EMF_ANTI_ALIASING, video::EAAM_OFF);
+		//material.setFlag(video::EMF_ANTI_ALIASING, video::EAAM_SIMPLE);
 		material.setFlag(video::EMF_FOG_ENABLE, true);
 
 		for(u32 i=0; i<fastfaces_new.size(); i++)
@@ -708,10 +714,13 @@ void MapBlock::updateMesh(u32 daynight_ratio)
 				continue;
 
 			material.setTexture(0, texture);
-			if(f.tile.alpha != 255)
+			
+			f.tile.applyMaterialOptions(material);
+			
+			/*if(f.tile.alpha != 255)
 				material.MaterialType = video::EMT_TRANSPARENT_VERTEX_ALPHA;
 			else
-				material.MaterialType = video::EMT_SOLID;
+				material.MaterialType = video::EMT_SOLID;*/
 			
 			collector.append(material, f.vertices, 4, indices, 6);
 		}
@@ -727,14 +736,22 @@ void MapBlock::updateMesh(u32 daynight_ratio)
 	//TimeTaker timer2("updateMesh() adding special stuff");
 
 	// Flowing water material
-	video::SMaterial material_w1;
-	material_w1.setFlag(video::EMF_LIGHTING, false);
-	material_w1.setFlag(video::EMF_BACK_FACE_CULLING, false);
-	material_w1.setFlag(video::EMF_BILINEAR_FILTER, false);
-	material_w1.setFlag(video::EMF_FOG_ENABLE, true);
-	material_w1.MaterialType = video::EMT_TRANSPARENT_VERTEX_ALPHA;
-	material_w1.setTexture(0,
-			g_irrlicht->getTexture("water.png"));
+	video::SMaterial material_water1;
+	material_water1.setFlag(video::EMF_LIGHTING, false);
+	material_water1.setFlag(video::EMF_BACK_FACE_CULLING, false);
+	material_water1.setFlag(video::EMF_BILINEAR_FILTER, false);
+	material_water1.setFlag(video::EMF_FOG_ENABLE, true);
+	material_water1.MaterialType = video::EMT_TRANSPARENT_VERTEX_ALPHA;
+	material_water1.setTexture(0, g_irrlicht->getTexture("water.png"));
+
+	// New-style leaves material
+	video::SMaterial material_leaves1;
+	material_leaves1.setFlag(video::EMF_LIGHTING, false);
+	//material_leaves1.setFlag(video::EMF_BACK_FACE_CULLING, false);
+	material_leaves1.setFlag(video::EMF_BILINEAR_FILTER, false);
+	material_leaves1.setFlag(video::EMF_FOG_ENABLE, true);
+	material_leaves1.MaterialType = video::EMT_TRANSPARENT_ALPHA_CHANNEL_REF;
+	material_leaves1.setTexture(0, g_irrlicht->getTexture("leaves.png"));
 
 	for(s16 z=0; z<MAP_BLOCKSIZE; z++)
 	for(s16 y=0; y<MAP_BLOCKSIZE; y++)
@@ -1020,7 +1037,7 @@ void MapBlock::updateMesh(u32 daynight_ratio)
 
 				u16 indices[] = {0,1,2,2,3,0};
 				// Add to mesh collector
-				collector.append(material_w1, vertices, 4, indices, 6);
+				collector.append(material_water1, vertices, 4, indices, 6);
 			}
 			
 			/*
@@ -1047,11 +1064,11 @@ void MapBlock::updateMesh(u32 daynight_ratio)
 
 				u16 indices[] = {0,1,2,2,3,0};
 				// Add to mesh collector
-				collector.append(material_w1, vertices, 4, indices, 6);
+				collector.append(material_water1, vertices, 4, indices, 6);
 			}
 		}
 		/*
-			Add water sources to mesh
+			Add water sources to mesh if using new style
 		*/
 		else if(n.d == CONTENT_WATERSOURCE && new_style_water)
 		{
@@ -1089,7 +1106,66 @@ void MapBlock::updateMesh(u32 daynight_ratio)
 
 			u16 indices[] = {0,1,2,2,3,0};
 			// Add to mesh collector
-			collector.append(material_w1, vertices, 4, indices, 6);
+			collector.append(material_water1, vertices, 4, indices, 6);
+		}
+		/*
+			Add leaves if using new style
+		*/
+		else if(n.d == CONTENT_LEAVES && new_style_leaves)
+		{
+			u8 l = decode_light(n.getLightBlend(daynight_ratio));
+			video::SColor c(255,l,l,l);
+
+			for(u32 j=0; j<6; j++)
+			{
+				video::S3DVertex vertices[4] =
+				{
+					video::S3DVertex(-BS/2,-BS/2,BS/2, 0,0,0, c, 0,1),
+					video::S3DVertex(BS/2,-BS/2,BS/2, 0,0,0, c, 1,1),
+					video::S3DVertex(BS/2,BS/2,BS/2, 0,0,0, c, 1,0),
+					video::S3DVertex(-BS/2,BS/2,BS/2, 0,0,0, c, 0,0),
+				};
+
+				if(j == 0)
+				{
+					for(u16 i=0; i<4; i++)
+						vertices[i].Pos.rotateXZBy(0);
+				}
+				else if(j == 1)
+				{
+					for(u16 i=0; i<4; i++)
+						vertices[i].Pos.rotateXZBy(180);
+				}
+				else if(j == 2)
+				{
+					for(u16 i=0; i<4; i++)
+						vertices[i].Pos.rotateXZBy(-90);
+				}
+				else if(j == 3)
+				{
+					for(u16 i=0; i<4; i++)
+						vertices[i].Pos.rotateXZBy(90);
+				}
+				else if(j == 4)
+				{
+					for(u16 i=0; i<4; i++)
+						vertices[i].Pos.rotateYZBy(-90);
+				}
+				else if(j == 5)
+				{
+					for(u16 i=0; i<4; i++)
+						vertices[i].Pos.rotateYZBy(90);
+				}
+
+				for(u16 i=0; i<4; i++)
+				{
+					vertices[i].Pos += intToFloat(p + getPosRelative());
+				}
+
+				u16 indices[] = {0,1,2,2,3,0};
+				// Add to mesh collector
+				collector.append(material_leaves1, vertices, 4, indices, 6);
+			}
 		}
 	}
 
diff --git a/src/mapnode.cpp b/src/mapnode.cpp
index d585238ae..e9a6ba904 100644
--- a/src/mapnode.cpp
+++ b/src/mapnode.cpp
@@ -40,6 +40,9 @@ ContentFeatures & content_features(u8 i)
 
 void init_mapnode(IIrrlichtWrapper *irrlicht)
 {
+	bool new_style_water = g_settings.getBool("new_style_water");
+	bool new_style_leaves = g_settings.getBool("new_style_leaves");
+	
 	u8 i;
 	ContentFeatures *f = NULL;
 
@@ -92,9 +95,25 @@ void init_mapnode(IIrrlichtWrapper *irrlicht)
 	
 	i = CONTENT_LEAVES;
 	f = &g_content_features[i];
-	f->setAllTextures(irrlicht->getTextureId("leaves.png"));
-	f->param_type = CPT_MINERAL;
+	f->light_propagates = true;
+	//f->param_type = CPT_MINERAL;
+	f->param_type = CPT_LIGHT;
 	f->is_ground_content = true;
+	if(new_style_leaves)
+	{
+		f->solidness = 0; // drawn separately, makes no faces
+	}
+	else
+	{
+		f->setAllTextures(irrlicht->getTextureId("leaves.png"));
+	}
+	/*{
+		TileSpec t;
+		t.spec = TextureSpec(irrlicht->getTextureId("leaves.png"));
+		//t.material_type = MATERIAL_ALPHA_SIMPLE;
+		//t.material_flags |= MATERIAL_FLAG_BACKFACE_CULLING;
+		f->setAllTiles(t);
+	}*/
 	
 	i = CONTENT_COALSTONE;
 	f = &g_content_features[i];
@@ -141,17 +160,26 @@ void init_mapnode(IIrrlichtWrapper *irrlicht)
 	f->buildable_to = true;
 	f->liquid_type = LIQUID_FLOWING;
 	
-	bool new_style_water = g_settings.getBool("new_style_water");
-	
 	i = CONTENT_WATERSOURCE;
 	f = &g_content_features[i];
-	//f->setTexture(0, irrlicht->getTextureId("water.png"), WATER_ALPHA);
-	if(new_style_water == false)
-		f->setAllTextures(irrlicht->getTextureId("water.png"), WATER_ALPHA);
 	f->setInventoryTexture(irrlicht->getTextureId("water.png"));
+	if(new_style_water)
+	{
+		f->solidness = 0; // drawn separately, makes no faces
+	}
+	else // old style
+	{
+		f->setAllTextures(irrlicht->getTextureId("water.png"), WATER_ALPHA);
+		TileSpec t;
+		t.spec = TextureSpec(irrlicht->getTextureId("water.png"));
+		t.alpha = WATER_ALPHA;
+		t.material_type = MATERIAL_ALPHA_VERTEX;
+		t.material_flags &= ~MATERIAL_FLAG_BACKFACE_CULLING;
+		f->setAllTiles(t);
+		f->solidness = 1;
+	}
 	f->param_type = CPT_LIGHT;
 	f->light_propagates = true;
-	f->solidness = 1;
 	f->walkable = false;
 	f->pointable = false;
 	f->diggable = false;
diff --git a/src/mapnode.h b/src/mapnode.h
index fcbb80fd1..0aaa4dc78 100644
--- a/src/mapnode.h
+++ b/src/mapnode.h
@@ -167,25 +167,38 @@ struct ContentFeatures
 
 	~ContentFeatures();
 	
+	// Quickhands for simple materials
+	void setTexture(u16 i, const TextureSpec &spec, u8 alpha=255)
+	{
+		tiles[i].spec = spec;
+		if(alpha != 255)
+		{
+			tiles[i].alpha = alpha;
+			tiles[i].material_type = MATERIAL_ALPHA_VERTEX;
+		}
+	}
 	void setAllTextures(const TextureSpec &spec, u8 alpha=255)
 	{
 		for(u16 i=0; i<6; i++)
 		{
-			tiles[i].spec = spec;
-			tiles[i].alpha = alpha;
+			setTexture(i, spec, alpha);
 		}
 		
 		// Set this too so it can be left as is most times
-		/*if(inventory_image_path == "")
-			inventory_image_path = porting::getDataPath(imgname.c_str());*/
-
 		if(inventory_texture.empty())
 			inventory_texture = spec;
 	}
-	void setTexture(u16 i, const TextureSpec &spec, u8 alpha=255)
+
+	void setTile(u16 i, const TileSpec &tile)
 	{
-		tiles[i].spec = spec;
-		tiles[i].alpha = alpha;
+		tiles[i] = tile;
+	}
+	void setAllTiles(const TileSpec &tile)
+	{
+		for(u16 i=0; i<6; i++)
+		{
+			setTile(i, tile);
+		}
 	}
 
 	void setInventoryTexture(const TextureSpec &spec)
@@ -417,12 +430,11 @@ struct MapNode
 	
 	union
 	{
-		u8 param2;
-
 		/*
-			Direction for torches and other stuff.
-			Format is freeform. e.g. packDir or encode_dirs can be used.
+			The second parameter. Initialized to 0.
+			Direction for torches and flowing water.
 		*/
+		u8 param2;
 		u8 dir;
 	};
 
diff --git a/src/modalMenu.h b/src/modalMenu.h
index 2323a7e42..ebbb06cfe 100644
--- a/src/modalMenu.h
+++ b/src/modalMenu.h
@@ -86,7 +86,10 @@ public:
 	}
 	
 	/*
-		This should be called when the menu wants to quit
+		This should be called when the menu wants to quit.
+
+		WARNING: THIS DEALLOCATES THE MENU FROM MEMORY. Return
+		immediately if you call this from the menu itself.
 	*/
 	void quitMenu()
 	{
diff --git a/src/player.cpp b/src/player.cpp
index 9817d590d..4619e870f 100644
--- a/src/player.cpp
+++ b/src/player.cpp
@@ -30,7 +30,11 @@ with this program; if not, write to the Free Software Foundation, Inc.,
 Player::Player():
 	touching_ground(false),
 	in_water(false),
+	in_water_stable(false),
+	swimming_up(false),
 	peer_id(PEER_ID_INEXISTENT),
+	m_pitch(0),
+	m_yaw(0),
 	m_speed(0,0,0),
 	m_position(0,0,0)
 {
@@ -275,11 +279,10 @@ void LocalPlayer::move(f32 dtime, Map &map)
 	position += m_speed * dtime;
 
 	bool free_move = g_settings.getBool("free_move");
-	bool terrain_viewer = g_settings.getBool("terrain_viewer");
 	
 	// Skip collision detection if player is non-local or
 	// a special movement mode is used
-	if(isLocal() == false || free_move || terrain_viewer)
+	if(isLocal() == false || free_move)
 	{
 		setPosition(position);
 		return;
@@ -292,7 +295,7 @@ void LocalPlayer::move(f32 dtime, Map &map)
 	v3s16 pos_i = floatToInt(position);
 	
 	/*
-		Check if player is in water
+		Check if player is in water (the oscillating value)
 	*/
 	try{
 		if(in_water)
@@ -311,6 +314,18 @@ void LocalPlayer::move(f32 dtime, Map &map)
 		in_water = false;
 	}
 
+	/*
+		Check if player is in water (the stable value)
+	*/
+	try{
+		v3s16 pp = floatToInt(position + v3f(0,0,0));
+		in_water_stable = content_liquid(map.getNode(pp).d);
+	}
+	catch(InvalidPositionException &e)
+	{
+		in_water_stable = false;
+	}
+
 	// The frame length is limited to the player going 0.1*BS per call
 	f32 d = (float)BS * 0.15;
 
@@ -526,10 +541,12 @@ void LocalPlayer::applyControl(float dtime)
 			speed.Y = 6.5*BS;
 			setSpeed(speed);
 		}
+		// Use the oscillating value for getting out of water
+		// (so that the player doesn't fly on the surface)
 		else if(in_water)
 		{
 			v3f speed = getSpeed();
-			speed.Y = 2.0*BS;
+			speed.Y = 1.5*BS;
 			setSpeed(speed);
 			swimming_up = true;
 		}
diff --git a/src/player.h b/src/player.h
index 3bbb4034d..51e2320bc 100644
--- a/src/player.h
+++ b/src/player.h
@@ -114,7 +114,10 @@ public:
 	void deSerialize(std::istream &is);
 
 	bool touching_ground;
+	// This oscillates so that the player jumps a bit above the surface
 	bool in_water;
+	// This is more stable and defines the maximum speed of the player
+	bool in_water_stable;
 	bool swimming_up;
 	
 	Inventory inventory;
diff --git a/src/porting.cpp b/src/porting.cpp
index a1e6fd02d..592636336 100644
--- a/src/porting.cpp
+++ b/src/porting.cpp
@@ -81,8 +81,9 @@ void initializePaths()
 		#include <unistd.h>
 	
 	char buf[BUFSIZ];
+	memset(buf, 0, BUFSIZ);
 	// Get path to executable
-	readlink("/proc/self/exe", buf, BUFSIZ);
+	readlink("/proc/self/exe", buf, BUFSIZ-1);
 	
 	pathRemoveFile(buf, '/');
 
@@ -144,8 +145,9 @@ void initializePaths()
 		#include <unistd.h>
 	
 	char buf[BUFSIZ];
+	memset(buf, 0, BUFSIZ);
 	// Get path to executable
-	readlink("/proc/self/exe", buf, BUFSIZ);
+	readlink("/proc/self/exe", buf, BUFSIZ-1);
 	
 	pathRemoveFile(buf, '/');
 
diff --git a/src/server.cpp b/src/server.cpp
index 71a094e14..59d9acee4 100644
--- a/src/server.cpp
+++ b/src/server.cpp
@@ -962,8 +962,8 @@ void PlayerInfo::PrintLine(std::ostream *s)
 {
 	(*s)<<id<<": ";
 	(*s)<<"\""<<name<<"\" ("
-			<<position.X<<","<<position.Y
-			<<","<<position.Z<<") ";
+			<<(position.X/10)<<","<<(position.Y/10)
+			<<","<<(position.Z/10)<<") ";
 	address.print(s);
 	(*s)<<" avg_rtt="<<avg_rtt;
 	(*s)<<std::endl;
@@ -2668,6 +2668,7 @@ void Server::SendPlayerInfos()
 				"peer_id="<<player->peer_id<<std::endl;*/
 		
 		writeU16(&data[start], player->peer_id);
+		memset((char*)&data[start+2], 0, PLAYERNAME_SIZE);
 		snprintf((char*)&data[start+2], PLAYERNAME_SIZE, "%s", player->getName());
 		start += 2+PLAYERNAME_SIZE;
 	}
@@ -3204,7 +3205,7 @@ Player *Server::emergePlayer(const char *name, const char *password,
 #if 1
 		player->setPosition(intToFloat(v3s16(
 				0,
-				40, //64,
+				45, //64,
 				0
 		)));
 #endif
diff --git a/src/tile.h b/src/tile.h
index ff495abc4..a68731f55 100644
--- a/src/tile.h
+++ b/src/tile.h
@@ -25,51 +25,69 @@ with this program; if not, write to the Free Software Foundation, Inc.,
 #include "texture.h"
 #include <string>
 
+enum MaterialType{
+	MATERIAL_ALPHA_NONE,
+	MATERIAL_ALPHA_VERTEX,
+	MATERIAL_ALPHA_SIMPLE, // >127 = opaque
+	MATERIAL_ALPHA_BLEND,
+};
+
+// Material flags
+#define MATERIAL_FLAG_BACKFACE_CULLING 0x01
+
+/*
+	This fully defines the looks of a tile.
+	The SMaterial of a tile is constructed according to this.
+*/
 struct TileSpec
 {
 	TileSpec():
-		alpha(255)
+		alpha(255),
+		material_type(MATERIAL_ALPHA_NONE),
+		material_flags(
+			MATERIAL_FLAG_BACKFACE_CULLING
+		)
 	{
 	}
 
 	bool operator==(TileSpec &other)
 	{
-		return (spec == other.spec && alpha == other.alpha);
+		return (
+			spec == other.spec &&
+			alpha == other.alpha &&
+			material_type == other.material_type &&
+			material_flags == other.material_flags
+		);
 	}
 	
-	TextureSpec spec;
-	u8 alpha;
-};
-
-#if 0
-struct TileSpec
-{
-	TileSpec():
-		alpha(255)
+	// Sets everything else except the texture in the material
+	void applyMaterialOptions(video::SMaterial &material)
 	{
-	}
+		if(alpha != 255 && material_type != MATERIAL_ALPHA_VERTEX)
+			dstream<<"WARNING: TileSpec: alpha != 255 "
+					"but not MATERIAL_ALPHA_VERTEX"
+					<<std::endl;
 
-	TileSpec(const std::string &a_name):
-		name(a_name),
-		alpha(255)
-	{
-	}
+		if(material_type == MATERIAL_ALPHA_NONE)
+			material.MaterialType = video::EMT_SOLID;
+		else if(material_type == MATERIAL_ALPHA_VERTEX)
+			material.MaterialType = video::EMT_TRANSPARENT_VERTEX_ALPHA;
+		else if(material_type == MATERIAL_ALPHA_SIMPLE)
+			material.MaterialType = video::EMT_TRANSPARENT_ALPHA_CHANNEL_REF;
+		else if(material_type == MATERIAL_ALPHA_BLEND)
+			material.MaterialType = video::EMT_TRANSPARENT_ALPHA_CHANNEL;
 
-	TileSpec(const char *a_name):
-		name(a_name),
-		alpha(255)
-	{
-	}
-
-	bool operator==(TileSpec &other)
-	{
-		return (name == other.name && alpha == other.alpha);
+		material.BackfaceCulling = (material_flags & MATERIAL_FLAG_BACKFACE_CULLING) ? true : false;
 	}
 	
-	// path + mods
-	std::string name;
+	// Specification of texture
+	TextureSpec spec;
+	// Vertex alpha
 	u8 alpha;
+	// Material type
+	u8 material_type;
+	// Material flags
+	u8 material_flags;
 };
-#endif
 
 #endif
diff --git a/src/utility.h b/src/utility.h
index 10b7af638..7d8102b62 100644
--- a/src/utility.h
+++ b/src/utility.h
@@ -1087,6 +1087,8 @@ public:
 			n = m_defaults.find(name);
 			if(n == NULL)
 			{
+				dstream<<"INFO: Settings: Setting not found: \""
+						<<name<<"\""<<std::endl;
 				throw SettingNotFoundException("Setting not found");
 			}
 		}