Fix double free

include/rewindow/types/window.h

@@ -322,7 +322,7 @@ public:
                      bool wResizable = true,
                      bool wVsync = false) :
                      RWindowImpl(wTitle, wWidth, wHeight, wRenderer, wFullscreen, wResizable, wVsync) {};
-    ~RWindow();
+    ~RWindow() = default;
 public:
     // Platform dependant
     void Open() { RWindowImpl::Open(); };

main.cpp

@@ -26,9 +26,9 @@ class MyWindow : public ReWindow::RWindow {
 	void OnKeyDown(const ReWindow::KeyDownEvent& e) override {}
 
 	void OnRefresh(float elapsed) override {
-            //glClearColor(255, 0, 0, 255);
-            //glClear(GL_COLOR_BUFFER_BIT);
-        //GLSwapBuffers();
+            glClearColor(255, 0, 0, 255);
+            glClear(GL_COLOR_BUFFER_BIT);
+        GLSwapBuffers();
 		auto pos = GetMouseCoordinates();
 		//std::cout << pos.x << ", " << pos.y << std::endl;
 
@@ -105,7 +105,7 @@ int main() {
     }
     */
 
-	auto* window = new MyWindow("Test Window", 600, 480);
+	MyWindow* window = new MyWindow("Test Window", 600, 480);
 	jlog::Debug(std::format("New window '{}' created. width={} height={}", window->GetTitle(), window->GetWidth(),
                             window->GetHeight()));
 
@@ -148,10 +148,11 @@ int main() {
         std::cout << window->GetMouseWheelPersistent() << std::endl;
     };
 
+    /*
 	while (!window->IsClosing()) {
         //window->PollEvents();
         window->ManagedRefresh();
-	}
+	}*/
 
     // Found problem
     // free(): double free detected in tcache 2
@@ -173,16 +174,20 @@ int main() {
 
     //exit(0);
 
-    auto* windowdos = new MyWindow("Test Window Dos", 600, 480);
+    MyWindow* windowdos = new MyWindow("Test Window Dos", 600, 480);
 
     windowdos->SetRenderer(ReWindow::RenderingAPI::OPENGL);
 
     windowdos->Open();
-
+/*
     while (!windowdos->IsClosing()) {
         //window->PollEvents();
+        window->ManagedRefresh();
         windowdos->ManagedRefresh();
     }
+    */
+    delete window;
+    delete windowdos;
     return 0;
 }
 

src/platform/linux/window.cpp

@@ -166,7 +166,7 @@ void RWindowImpl::PollEvents() {
         if (pPlatform->xev.type == ClientMessage)
             Logger::Info(std::format("Event '{}'", "ClientMessage"));
 
-        // Hmm
+        // Hmmm
         /*
         if (pPlatform->xev.xclient.message_type == XInternAtom(pPlatform->display, "WM_PROTOCOLS", False) && static_cast<Atom>(pPlatform->xev.xclient.data.l[0]) == pPlatform->wmDeleteWindow) {
             Close();

src/platform/shared/window.cpp

@@ -42,13 +42,13 @@ flags{false,wFullscreen,wResizable,wVsync} {
  */
 
 
-RWindow::~RWindow() {
+//RWindow::~RWindow() {
     /*
     if (open)
         DestroyOSWindowHandle();
     */
-   RWindowImpl::~RWindowImpl();
-}
+   //RWindowImpl::~RWindowImpl();
+//}