Fix a crash or random memory leak when reseting saved environment variable in test_se...
[oweals/minetest.git] / src / sky.cpp
index 211a2dcdcd9f78656fd1095ab8335c9ead281a37..02e882592b9ef35df75467a28b0b86eb2c0f9802 100644 (file)
@@ -8,21 +8,14 @@
 #include "profiler.h"
 #include "util/numeric.h"
 #include <cmath>
+#include "client/renderingengine.h"
 #include "settings.h"
 #include "camera.h"  // CameraModes
 
 
-Sky::Sky(scene::ISceneNode* parent, scene::ISceneManager* mgr, s32 id,
-               ITextureSource *tsrc):
-               scene::ISceneNode(parent, mgr, id),
-               m_visible(true),
-               m_fallback_bg_color(255, 255, 255, 255),
-               m_first_update(true),
-               m_brightness(0.5),
-               m_cloud_brightness(0.5),
-               m_bgcolor_bright_f(1, 1, 1, 1),
-               m_skycolor_bright_f(1, 1, 1, 1),
-               m_cloudcolor_bright_f(1, 1, 1, 1)
+Sky::Sky(s32 id, ITextureSource *tsrc):
+               scene::ISceneNode(RenderingEngine::get_scene_manager()->getRootSceneNode(),
+                       RenderingEngine::get_scene_manager(), id)
 {
        setAutomaticCulling(scene::EAC_OFF);
        m_box.MaxEdge.set(0, 0, 0);
@@ -75,13 +68,13 @@ Sky::Sky(scene::ISceneNode* parent, scene::ISceneManager* mgr, s32 id,
                        m_materials[4].Lighting = true;
        }
 
-       for (u32 i = 0; i < SKY_STAR_COUNT; i++) {
-               m_stars[i] = v3f(
+       for (v3f &star : m_stars) {
+               star = v3f(
                        myrand_range(-10000, 10000),
                        myrand_range(-10000, 10000),
                        myrand_range(-10000, 10000)
                );
-               m_stars[i].normalize();
+               star.normalize();
        }
 
        m_directional_colored_fog = g_settings->getBool("directional_colored_fog");
@@ -107,7 +100,7 @@ void Sky::render()
 
        if (!camera || !driver)
                return;
-       
+
        ScopeProfiler sp(g_profiler, "Sky::render()", SPT_AVG);
 
        // Draw perspective skybox
@@ -136,7 +129,7 @@ void Sky::render()
                float moonsize = 0.04;
                video::SColorf mooncolor_f(0.50, 0.57, 0.65, 1);
                video::SColorf mooncolor2_f(0.85, 0.875, 0.9, 1);
-               
+
                float nightlength = 0.415;
                float wn = nightlength / 2;
                float wicked_time_of_day = 0;
@@ -179,11 +172,11 @@ void Sky::render()
                const f32 o = 0.0f;
                static const u16 indices[4] = {0, 1, 2, 3};
                video::S3DVertex vertices[4];
-               
+
                driver->setMaterial(m_materials[1]);
-               
+
                video::SColor cloudyfogcolor = m_bgcolor;
-               
+
                // Draw far cloudy fog thing blended with skycolor
                for (u32 j = 0; j < 4; j++) {
                        video::SColor c = cloudyfogcolor.getInterpolated(m_skycolor, 0.45);
@@ -191,19 +184,19 @@ void Sky::render()
                        vertices[1] = video::S3DVertex( 1, 0.08, -1, 0, 0, 1, c, o, t);
                        vertices[2] = video::S3DVertex( 1, 0.12, -1, 0, 0, 1, c, o, o);
                        vertices[3] = video::S3DVertex(-1, 0.12, -1, 0, 0, 1, c, t, o);
-                       for (u32 i = 0; i < 4; i++) {
+                       for (video::S3DVertex &vertex : vertices) {
                                if (j == 0)
                                        // Don't switch
                                        {}
                                else if (j == 1)
                                        // Switch from -Z (south) to +X (east)
-                                       vertices[i].Pos.rotateXZBy(90);
+                                       vertex.Pos.rotateXZBy(90);
                                else if (j == 2)
                                        // Switch from -Z (south) to -X (west)
-                                       vertices[i].Pos.rotateXZBy(-90);
+                                       vertex.Pos.rotateXZBy(-90);
                                else
                                        // Switch from -Z (south) to +Z (north)
-                                       vertices[i].Pos.rotateXZBy(-180);
+                                       vertex.Pos.rotateXZBy(-180);
                        }
                        driver->drawIndexedTriangleFan(&vertices[0], 4, indices, 2);
                }
@@ -215,19 +208,19 @@ void Sky::render()
                        vertices[1] = video::S3DVertex( 1, -1.0, -1, 0, 0, 1, c, o, t);
                        vertices[2] = video::S3DVertex( 1, 0.08, -1, 0, 0, 1, c, o, o);
                        vertices[3] = video::S3DVertex(-1, 0.08, -1, 0, 0, 1, c, t, o);
-                       for (u32 i = 0; i < 4; i++) {
+                       for (video::S3DVertex &vertex : vertices) {
                                if (j == 0)
                                        // Don't switch
                                        {}
                                else if (j == 1)
                                        // Switch from -Z (south) to +X (east)
-                                       vertices[i].Pos.rotateXZBy(90);
+                                       vertex.Pos.rotateXZBy(90);
                                else if (j == 2)
                                        // Switch from -Z (south) to -X (west)
-                                       vertices[i].Pos.rotateXZBy(-90);
+                                       vertex.Pos.rotateXZBy(-90);
                                else
                                        // Switch from -Z (south) to +Z (north)
-                                       vertices[i].Pos.rotateXZBy(-180);
+                                       vertex.Pos.rotateXZBy(-180);
                        }
                        driver->drawIndexedTriangleFan(&vertices[0], 4, indices, 2);
                }
@@ -240,6 +233,10 @@ void Sky::render()
                vertices[3] = video::S3DVertex(-1, -1.0, 1, 0, 1, 0, c, t, o);
                driver->drawIndexedTriangleFan(&vertices[0], 4, indices, 2);
 
+               // If sun, moon and stars are (temporarily) disabled, abort here
+               if (!m_bodies_visible)
+                       return;
+
                driver->setMaterial(m_materials[2]);
 
                // Draw sunrise/sunset horizon glow texture (textures/base/pack/sunrisebg.png)
@@ -255,13 +252,13 @@ void Sky::render()
                        vertices[1] = video::S3DVertex( 1, -0.05 + y, -1, 0, 0, 1, c, o, t);
                        vertices[2] = video::S3DVertex( 1,   0.2 + y, -1, 0, 0, 1, c, o, o);
                        vertices[3] = video::S3DVertex(-1,   0.2 + y, -1, 0, 0, 1, c, t, o);
-                       for (u32 i = 0; i < 4; i++) {
+                       for (video::S3DVertex &vertex : vertices) {
                                if (wicked_time_of_day < 0.5)
                                        // Switch from -Z (south) to +X (east)
-                                       vertices[i].Pos.rotateXZBy(90);
+                                       vertex.Pos.rotateXZBy(90);
                                else
                                        // Switch from -Z (south) to -X (west)
-                                       vertices[i].Pos.rotateXZBy(-90);
+                                       vertex.Pos.rotateXZBy(-90);
                        }
                        driver->drawIndexedTriangleFan(&vertices[0], 4, indices, 2);
                }
@@ -277,10 +274,10 @@ void Sky::render()
                                vertices[1] = video::S3DVertex( d, -d, -1, 0, 0, 1, c, o, t);
                                vertices[2] = video::S3DVertex( d,  d, -1, 0, 0, 1, c, o, o);
                                vertices[3] = video::S3DVertex(-d,  d, -1, 0, 0, 1, c, t, o);
-                               for (u32 i = 0; i < 4; i++) {
+                               for (video::S3DVertex &vertex : vertices) {
                                        // Switch from -Z (south) to +X (east)
-                                       vertices[i].Pos.rotateXZBy(90);
-                                       vertices[i].Pos.rotateXYBy(wicked_time_of_day * 360 - 90);
+                                       vertex.Pos.rotateXZBy(90);
+                                       vertex.Pos.rotateXYBy(wicked_time_of_day * 360 - 90);
                                }
                                driver->drawIndexedTriangleFan(&vertices[0], 4, indices, 2);
 
@@ -291,10 +288,10 @@ void Sky::render()
                                vertices[1] = video::S3DVertex( d, -d, -1, 0, 0, 1, c, o, t);
                                vertices[2] = video::S3DVertex( d,  d, -1, 0, 0, 1, c, o, o);
                                vertices[3] = video::S3DVertex(-d,  d, -1, 0, 0, 1, c, t, o);
-                               for (u32 i = 0; i < 4; i++) {
+                               for (video::S3DVertex &vertex : vertices) {
                                        // Switch from -Z (south) to +X (east)
-                                       vertices[i].Pos.rotateXZBy(90);
-                                       vertices[i].Pos.rotateXYBy(wicked_time_of_day * 360 - 90);
+                                       vertex.Pos.rotateXZBy(90);
+                                       vertex.Pos.rotateXYBy(wicked_time_of_day * 360 - 90);
                                }
                                driver->drawIndexedTriangleFan(&vertices[0], 4, indices, 2);
 
@@ -303,10 +300,10 @@ void Sky::render()
                                vertices[1] = video::S3DVertex( d, -d, -1, 0, 0, 1, suncolor, o, t);
                                vertices[2] = video::S3DVertex( d,  d, -1, 0, 0, 1, suncolor, o, o);
                                vertices[3] = video::S3DVertex(-d,  d, -1, 0, 0, 1, suncolor, t, o);
-                               for (u32 i = 0; i < 4; i++) {
+                               for (video::S3DVertex &vertex : vertices) {
                                        // Switch from -Z (south) to +X (east)
-                                       vertices[i].Pos.rotateXZBy(90);
-                                       vertices[i].Pos.rotateXYBy(wicked_time_of_day * 360 - 90);
+                                       vertex.Pos.rotateXZBy(90);
+                                       vertex.Pos.rotateXYBy(wicked_time_of_day * 360 - 90);
                                }
                                driver->drawIndexedTriangleFan(&vertices[0], 4, indices, 2);
 
@@ -315,10 +312,10 @@ void Sky::render()
                                vertices[1] = video::S3DVertex( d, -d, -1, 0, 0, 1, suncolor2, o, t);
                                vertices[2] = video::S3DVertex( d,  d, -1, 0, 0, 1, suncolor2, o, o);
                                vertices[3] = video::S3DVertex(-d,  d, -1, 0, 0, 1, suncolor2, t, o);
-                               for (u32 i = 0; i < 4; i++) {
+                               for (video::S3DVertex &vertex : vertices) {
                                        // Switch from -Z (south) to +X (east)
-                                       vertices[i].Pos.rotateXZBy(90);
-                                       vertices[i].Pos.rotateXYBy(wicked_time_of_day * 360 - 90);
+                                       vertex.Pos.rotateXZBy(90);
+                                       vertex.Pos.rotateXYBy(wicked_time_of_day * 360 - 90);
                                }
                                driver->drawIndexedTriangleFan(&vertices[0], 4, indices, 2);
                        } else {
@@ -333,10 +330,10 @@ void Sky::render()
                                vertices[1] = video::S3DVertex( d, -d, -1, 0, 0, 1, c, o, t);
                                vertices[2] = video::S3DVertex( d,  d, -1, 0, 0, 1, c, o, o);
                                vertices[3] = video::S3DVertex(-d,  d, -1, 0, 0, 1, c, t, o);
-                               for(u32 i = 0; i < 4; i++) {
+                               for (video::S3DVertex &vertex : vertices) {
                                        // Switch from -Z (south) to +X (east)
-                                       vertices[i].Pos.rotateXZBy(90);
-                                       vertices[i].Pos.rotateXYBy(wicked_time_of_day * 360 - 90);
+                                       vertex.Pos.rotateXZBy(90);
+                                       vertex.Pos.rotateXYBy(wicked_time_of_day * 360 - 90);
                                }
                                driver->drawIndexedTriangleFan(&vertices[0], 4, indices, 2);
                        }
@@ -353,13 +350,13 @@ void Sky::render()
                                vertices[1] = video::S3DVertex( d, -d, -1, 0, 0, 1, c, o, t);
                                vertices[2] = video::S3DVertex( d,  d, -1, 0, 0, 1, c, o, o);
                                vertices[3] = video::S3DVertex(-d,  d, -1, 0, 0, 1, c, t, o);
-                               for (u32 i = 0; i < 4; i++) {
+                               for (video::S3DVertex &vertex : vertices) {
                                        // Switch from -Z (south) to -X (west)
-                                       vertices[i].Pos.rotateXZBy(-90);
-                                       vertices[i].Pos.rotateXYBy(wicked_time_of_day * 360 - 90);
+                                       vertex.Pos.rotateXZBy(-90);
+                                       vertex.Pos.rotateXYBy(wicked_time_of_day * 360 - 90);
                                }
                                driver->drawIndexedTriangleFan(&vertices[0], 4, indices, 2);
-                       
+
                                d = moonsize * 1.3;
                                c = mooncolor;
                                c.setAlpha(0.15 * 255);
@@ -367,10 +364,10 @@ void Sky::render()
                                vertices[1] = video::S3DVertex( d, -d, -1, 0, 0, 1, c, o, t);
                                vertices[2] = video::S3DVertex( d,  d, -1, 0, 0, 1, c, o, o);
                                vertices[3] = video::S3DVertex(-d,  d, -1, 0, 0, 1, c, t, o);
-                               for (u32 i = 0; i < 4; i++) {
+                               for (video::S3DVertex &vertex : vertices) {
                                        // Switch from -Z (south) to -X (west)
-                                       vertices[i].Pos.rotateXZBy(-90);
-                                       vertices[i].Pos.rotateXYBy(wicked_time_of_day * 360 - 90);
+                                       vertex.Pos.rotateXZBy(-90);
+                                       vertex.Pos.rotateXYBy(wicked_time_of_day * 360 - 90);
                                }
                                driver->drawIndexedTriangleFan(&vertices[0], 4, indices, 2);
 
@@ -379,10 +376,10 @@ void Sky::render()
                                vertices[1] = video::S3DVertex( d, -d, -1, 0, 0, 1, mooncolor, o, t);
                                vertices[2] = video::S3DVertex( d,  d, -1, 0, 0, 1, mooncolor, o, o);
                                vertices[3] = video::S3DVertex(-d,  d, -1, 0, 0, 1, mooncolor, t, o);
-                               for (u32 i = 0; i < 4; i++) {
+                               for (video::S3DVertex &vertex : vertices) {
                                        // Switch from -Z (south) to -X (west)
-                                       vertices[i].Pos.rotateXZBy(-90);
-                                       vertices[i].Pos.rotateXYBy(wicked_time_of_day * 360 - 90);
+                                       vertex.Pos.rotateXZBy(-90);
+                                       vertex.Pos.rotateXYBy(wicked_time_of_day * 360 - 90);
                                }
                                driver->drawIndexedTriangleFan(&vertices[0], 4, indices, 2);
 
@@ -391,10 +388,10 @@ void Sky::render()
                                vertices[1] = video::S3DVertex( d2,-d,  -1, 0, 0, 1, mooncolor2, o, t);
                                vertices[2] = video::S3DVertex( d2, d2, -1, 0, 0, 1, mooncolor2, o, o);
                                vertices[3] = video::S3DVertex(-d,  d2, -1, 0, 0, 1, mooncolor2, t, o);
-                               for (u32 i = 0; i < 4; i++) {
+                               for (video::S3DVertex &vertex : vertices) {
                                        // Switch from -Z (south) to -X (west)
-                                       vertices[i].Pos.rotateXZBy(-90);
-                                       vertices[i].Pos.rotateXYBy(wicked_time_of_day * 360 - 90);
+                                       vertex.Pos.rotateXZBy(-90);
+                                       vertex.Pos.rotateXYBy(wicked_time_of_day * 360 - 90);
                                }
                                driver->drawIndexedTriangleFan(&vertices[0], 4, indices, 2);
                        } else {
@@ -409,18 +406,18 @@ void Sky::render()
                                vertices[1] = video::S3DVertex( d, -d, -1, 0, 0, 1, c, o, t);
                                vertices[2] = video::S3DVertex( d,  d, -1, 0, 0, 1, c, o, o);
                                vertices[3] = video::S3DVertex(-d,  d, -1, 0, 0, 1, c, t, o);
-                               for (u32 i = 0; i < 4; i++) {
+                               for (video::S3DVertex &vertex : vertices) {
                                        // Switch from -Z (south) to -X (west)
-                                       vertices[i].Pos.rotateXZBy(-90);
-                                       vertices[i].Pos.rotateXYBy(wicked_time_of_day * 360 - 90);
+                                       vertex.Pos.rotateXZBy(-90);
+                                       vertex.Pos.rotateXYBy(wicked_time_of_day * 360 - 90);
                                }
                                driver->drawIndexedTriangleFan(&vertices[0], 4, indices, 2);
                        }
                }
 
                // Draw stars
-               driver->setMaterial(m_materials[1]);
                do {
+                       driver->setMaterial(m_materials[1]);
                        float starbrightness = MYMAX(0, MYMIN(1,
                                (0.285 - fabs(wicked_time_of_day < 0.5 ?
                                wicked_time_of_day : (1.0 - wicked_time_of_day))) * 10));
@@ -463,8 +460,8 @@ void Sky::render()
                        driver->drawVertexPrimitiveList(vertices, SKY_STAR_COUNT * 4,
                                indices, SKY_STAR_COUNT, video::EVT_STANDARD,
                                scene::EPT_QUADS, video::EIT_16BIT);
-               } while(0);
-               
+               } while(false);
+
                // Draw far cloudy fog thing below east and west horizons
                for (u32 j = 0; j < 2; j++) {
                        video::SColor c = cloudyfogcolor;
@@ -472,14 +469,14 @@ void Sky::render()
                        vertices[1] = video::S3DVertex( 1, -1.0,  -1, 0, 0, 1, c, o, t);
                        vertices[2] = video::S3DVertex( 1, -0.02, -1, 0, 0, 1, c, o, o);
                        vertices[3] = video::S3DVertex(-1, -0.02, -1, 0, 0, 1, c, t, o);
-                       for (u32 i = 0; i < 4; i++) {
+                       for (video::S3DVertex &vertex : vertices) {
                                //if (wicked_time_of_day < 0.5)
                                if (j == 0)
                                        // Switch from -Z (south) to +X (east)
-                                       vertices[i].Pos.rotateXZBy(90);
+                                       vertex.Pos.rotateXZBy(90);
                                else
                                        // Switch from -Z (south) to -X (west)
-                                       vertices[i].Pos.rotateXZBy(-90);
+                                       vertex.Pos.rotateXZBy(-90);
                        }
                        driver->drawIndexedTriangleFan(&vertices[0], 4, indices, 2);
                }
@@ -508,7 +505,8 @@ void Sky::update(float time_of_day, float time_brightness,
        m_time_of_day = time_of_day;
        m_time_brightness = time_brightness;
        m_sunlight_seen = sunlight_seen;
-       
+       m_bodies_visible = true;
+
        bool is_dawn = (time_brightness >= 0.20 && time_brightness < 0.35);
 
        /*
@@ -533,9 +531,11 @@ void Sky::update(float time_of_day, float time_brightness,
        video::SColorf skycolor_bright_normal_f = video::SColor(255, 140, 186, 250);
        video::SColorf skycolor_bright_dawn_f = video::SColor(255, 180, 186, 250);
        video::SColorf skycolor_bright_night_f = video::SColor(255, 0, 107, 255);
-       
-       video::SColorf cloudcolor_bright_normal_f = video::SColor(255, 240, 240, 255);
-       video::SColorf cloudcolor_bright_dawn_f = video::SColor(255, 255, 223, 191);
+
+       // pure white: becomes "diffuse light component" for clouds
+       video::SColorf cloudcolor_bright_normal_f = video::SColor(255, 255, 255, 255);
+       // dawn-factoring version of pure white (note: R is above 1.0)
+       video::SColorf cloudcolor_bright_dawn_f(255.0f/240.0f, 223.0f/240.0f, 191.0f/255.0f);
 
        float cloud_color_change_fraction = 0.95;
        if (sunlight_seen) {
@@ -551,7 +551,7 @@ void Sky::update(float time_of_day, float time_brightness,
                else
                        m_brightness = m_brightness * 0.98 + direct_brightness * 0.02;
        }
-       
+
        m_clouds_visible = true;
        float color_change_fraction = 0.98;
        if (sunlight_seen) {
@@ -605,7 +605,7 @@ void Sky::update(float time_of_day, float time_brightness,
        );
 
        // Horizon coloring based on sun and moon direction during sunset and sunrise
-       video::SColor pointcolor = video::SColor(255, 255, 255, m_bgcolor.getAlpha());
+       video::SColor pointcolor = video::SColor(m_bgcolor.getAlpha(), 255, 255, 255);
        if (m_directional_colored_fog) {
                if (m_horizon_blend() != 0) {
                        // Calculate hemisphere value from yaw, (inverted in third person front view)