Add texture lazy loading to fix texture copying, update weather to use new lazy loading

src/panel/panel.hpp

@@ -29,5 +29,6 @@ namespace dashboard::panel {
 
     protected:
         SDL_Texture* _texture;
+        virtual void initTexture() = 0;
     };
 }

src/panel/weather.cpp

@@ -15,21 +15,19 @@ using namespace dashboard::panel;
 
 weather::weather(){
     std::cerr << "WEATHER CONSTRUCTOR\n";
+    //std::cerr << "Current Renderer : " << board::getRenderer() << "\n";
     _time_on_screen = WEATHER_DEFAULT_ON_SCREEN_TIME;
     _update_interval = std::chrono::milliseconds{UPDATE_INTERVAL};
-    _last_update = std::chrono::high_resolution_clock::now();
+    //let set to default, will make it so it updates the texture ASAP
+    //_last_update;
     _rss = rss_utils::rss(WEATHER_URL_SOURCE);
-
-    _texture = SDL_CreateTexture(board::getRenderer(),
-            SDL_PIXELFORMAT_RGBA8888, 
-            SDL_TEXTUREACCESS_TARGET, 
-            SCREEN_WIDTH, SCREEN_HEIGHT);
-
-    update();
 }
 
 weather::~weather(){
     std::cerr << "WEATHER DECONSTRUCTOR\n";
+    if(_texture != nullptr){
+        SDL_DestroyTexture(_texture);
+    }
 }
 
 ///////////////////////////////////////////////////////////////////////////////
@@ -37,19 +35,24 @@ weather::~weather(){
 ///////////////////////////////////////////////////////////////////////////////
 
 void weather::draw(){
+    std::cerr << "WEATHER::DRAW\n";
+
+    //create the texture if this is the first time running draw
+    if(_texture == nullptr)
+        initTexture();
+
     //check if its time to update
     if((std::chrono::high_resolution_clock::now() - _last_update) 
             > _update_interval){
+        std::cerr << "UPDATING WEATHER IF STATEMENT\n";
         //TODO multithread this
         update();
 
         update_texture();
     }
 
-    static SDL_Rect tgt = { 0, 0, SCREEN_WIDTH, SCREEN_HEIGHT };
+    SDL_RenderCopy(board::getRenderer(), _texture, NULL, NULL);
 
-    //SDL_RenderCopy(board::getRenderer(), _texture, NULL, 
-            //&tgt);
 
     //TODO add this all to one texture
 
@@ -68,7 +71,6 @@ void weather::update() {
 
     //fetch updates
     _rss.update();
-    
 }
 
 ///////////////////////////////////////
@@ -76,18 +78,49 @@ void weather::update() {
 // displayed, based on data in 
 // _rss
 void weather::update_texture(){
-    //SDL_SetRenderTarget(board::getRenderer(), _texture);
+    std::cerr << "WEATHER::UPDATE_TEXTURE\n";
+
+    int ret = SDL_SetRenderTarget(board::getRenderer(), _texture);
+    std::cerr << "ret : " << ret << "\n";
+    std::cerr << "Renderer : " << board::getRenderer() << "\n";
+    if(ret != 0)
+        SDL_Log("ERROR : %s\n", SDL_GetError());
+
 
     SDL_Rect tgt;
+
+    //title
     tgt.x = 50;
     tgt.y = 50;
     TTF_SizeText(board::getFont({ "Roboto_Mono/RobotoMono-Medium.ttf", 24 }),
             _rss.getTitle().c_str(),
             &tgt.w, &tgt.h);
-    
-    SDL_RenderCopy(board::getRenderer(), 
-            board::getString(_rss.getTitle(), 
-                { "Roboto_Mono/RobotoMono-Medium.ttf", 24 }), NULL, &tgt );
 
-    //SDL_SetRenderTarget(board::getRenderer(), NULL);
+    std::cerr << "tgt.w : " << tgt.w << "\n";
+    std::cerr << "tgt.h : " << tgt.h << "\n";
+    std::cerr << "board::getString : " << board::getString(_rss.getTitle(),{ "Roboto_Mono/RobotoMono-Medium.ttf", 24 }) << "\n";
+
+    ret = SDL_RenderCopy(board::getRenderer(), 
+            //board::getString(_rss.getTitle(), 
+            board::getString(_rss.getTitle(), 
+                { "Roboto_Mono/RobotoMono-Medium.ttf", 24 }), NULL, &tgt);
+
+    std::cerr << "ret : " << ret << "\n";
+
+    SDL_SetRenderTarget(board::getRenderer(), NULL);
+}
+
+///////////////////////////////////////
+// Lazy load the texture object
+// This is to make sure that all of SDL
+// is init before we attempt to draw anything
+void weather::initTexture(){
+    if(_texture == nullptr){
+        _texture = SDL_CreateTexture(board::getRenderer(),
+                SDL_PIXELFORMAT_RGBA8888, 
+                SDL_TEXTUREACCESS_TARGET, 
+                SCREEN_WIDTH, SCREEN_HEIGHT);
+
+        SDL_SetTextureBlendMode(_texture, SDL_BLENDMODE_NONE);
+    }
 }

src/panel/weather.hpp

@@ -28,6 +28,7 @@ namespace dashboard::panel {
     private:
         void update();
         void update_texture();
+        void initTexture();
 
         rss_utils::rss _rss;
         std::chrono::time_point<std::chrono::high_resolution_clock> _last_update;

src/panel/weather_config.hpp

@@ -16,5 +16,6 @@ namespace dashboard::panel {
 
     //How long should we wait between updates? in ms
     //Default 1 hour
-    constexpr size_t UPDATE_INTERVAL = 3600000;
+    //constexpr size_t UPDATE_INTERVAL = 3600000;
+    constexpr size_t UPDATE_INTERVAL = 5000;
 }