Re-usable WaterfallPanel to replace WaterfallContext

CMakeLists.txt

@@ -245,6 +245,8 @@ SET (cubicsdr_sources
 	src/util/GLFont.cpp
 	src/util/DataTree.cpp
     src/panel/ScopePanel.cpp
+    src/panel/SpectrumPanel.cpp
+    src/panel/WaterfallPanel.cpp
 	src/visual/ColorTheme.cpp
 	src/visual/PrimaryGLContext.cpp
 	src/visual/InteractiveCanvas.cpp
@@ -259,7 +261,6 @@ SET (cubicsdr_sources
 	src/visual/SpectrumCanvas.cpp
 	src/visual/SpectrumContext.cpp
 	src/visual/WaterfallCanvas.cpp
-	src/visual/WaterfallContext.cpp
 	src/process/VisualProcessor.cpp
 	src/process/ScopeVisualProcessor.cpp
 	src/process/SpectrumVisualProcessor.cpp
@@ -298,6 +299,8 @@ SET (cubicsdr_headers
 	src/util/GLFont.h
 	src/util/DataTree.h
     src/panel/ScopePanel.h
+    src/panel/SpectrumPanel.h
+    src/panel/WaterfallPanel.h
 	src/visual/ColorTheme.h
 	src/visual/PrimaryGLContext.h
 	src/visual/InteractiveCanvas.h
@@ -312,7 +315,6 @@ SET (cubicsdr_headers
 	src/visual/SpectrumCanvas.h
 	src/visual/SpectrumContext.h
 	src/visual/WaterfallCanvas.h
-	src/visual/WaterfallContext.h
 	src/process/VisualProcessor.h
 	src/process/ScopeVisualProcessor.h
 	src/process/SpectrumVisualProcessor.h

src/panel/SpectrumPanel.cpp

@@ -0,0 +1,2 @@
+#include "SpectrumPanel.h"
+

src/panel/SpectrumPanel.h

@@ -0,0 +1,7 @@
+#pragma once
+
+#include "GLPanel.h"
+
+class SpectrumPanel : public GLPanel {
+    
+};

src/panel/WaterfallPanel.cpp

@@ -1,18 +1,19 @@
-#include "WaterfallContext.h"
+#include "WaterfallPanel.h"
-#include "WaterfallCanvas.h"
-#include "CubicSDR.h"
 
-WaterfallContext::WaterfallContext(WaterfallCanvas *canvas, wxGLContext *sharedContext) :
+WaterfallPanel::WaterfallPanel() : GLPanel(), fft_size(0), waterfall_lines(0), waterfall_slice(NULL), activeTheme(NULL) {
-        PrimaryGLContext(canvas, sharedContext), fft_size(0), waterfall_lines(0), waterfall_slice(NULL), activeTheme(NULL) {
     for (int i = 0; i < 2; i++) {
         waterfall[i] = 0;
     }
 }
 
-void WaterfallContext::Setup(int fft_size_in, int num_waterfall_lines_in) {
+void WaterfallPanel::setup(int fft_size_in, int num_waterfall_lines_in) {
     waterfall_lines = num_waterfall_lines_in;
     fft_size = fft_size_in;
     
+    if (points.size() != fft_size) {
+        points.resize(fft_size);
+    }
+    
     for (int i = 0; i < 2; i++) {
         if (waterfall[i]) {
             glDeleteTextures(1, &waterfall[i]);
@@ -23,7 +24,7 @@ void WaterfallContext::Setup(int fft_size_in, int num_waterfall_lines_in) {
     }
 }
 
-void WaterfallContext::refreshTheme() {
+void WaterfallPanel::refreshTheme() {
     glEnable (GL_TEXTURE_2D);
     
     for (int i = 0; i < 2; i++) {
@@ -36,13 +37,20 @@ void WaterfallContext::refreshTheme() {
     }
 }
 
-void WaterfallContext::Draw(std::vector<float> &points, bool step) {
+void WaterfallPanel::setPoints(std::vector<float> &points) {
+    int halfPts = points.size()/2;
+    if (halfPts == fft_size) {
+        for (int i = 0; i < fft_size; i++) {
+            this->points[i] = points[i*2+1];
+        }
+    } else {
+        this->points.assign(points.begin(), points.end());
+    }
+}
 
+void WaterfallPanel::step() {
     int half_fft_size = fft_size / 2;
 
-    glEnable (GL_TEXTURE_2D);
-    glTexEnvf(GL_TEXTURE_ENV, GL_TEXTURE_ENV_MODE, GL_DECAL);
-
     if (!waterfall[0]) {
         glGenTextures(2, waterfall);
         
@@ -64,23 +72,18 @@ void WaterfallContext::Draw(std::vector<float> &points, bool step) {
             glTexImage2D(GL_TEXTURE_2D, 0, GL_RGB, half_fft_size, waterfall_lines, 0, GL_COLOR_INDEX, GL_UNSIGNED_BYTE, (GLvoid *) waterfall_tex);
         }
         
-		if (waterfall_slice != NULL) {
+        if (waterfall_slice != NULL) {
-			delete waterfall_slice;
+            delete waterfall_slice;
-		}
+        }
-		waterfall_slice = new unsigned char[half_fft_size];
+        waterfall_slice = new unsigned char[half_fft_size];
         
         delete[] waterfall_tex;
     }
     
-    if (activeTheme != ThemeMgr::mgr.currentTheme) {
-        refreshTheme();
-        activeTheme = ThemeMgr::mgr.currentTheme;
-    }
-
     if (points.size()) {
         for (int j = 0; j < 2; j++) {
             for (int i = 0, iMax = half_fft_size; i < iMax; i++) {
-                float v = points[(j * half_fft_size + i) * 2 + 1];
+                float v = points[j * half_fft_size + i];
                 
                 float wv = v < 0 ? 0 : (v > 0.99 ? 0.99 : v);
                 
@@ -97,7 +100,24 @@ void WaterfallContext::Draw(std::vector<float> &points, bool step) {
             waterfall_ofs[j]--;
         }
     }
+}
 
+void WaterfallPanel::drawPanelContents() {
+    if (!waterfall[0]) {
+        return;
+    }
+        
+    int half_fft_size = fft_size / 2;
+    
+    glLoadMatrixf(transform);
+    
+    glEnable (GL_TEXTURE_2D);
+    glTexEnvf(GL_TEXTURE_ENV, GL_TEXTURE_ENV_MODE, GL_DECAL);
+    
+    if (activeTheme != ThemeMgr::mgr.currentTheme) {
+        refreshTheme();
+        activeTheme = ThemeMgr::mgr.currentTheme;
+    }
     glColor3f(1.0, 1.0, 1.0);
     
     GLint vp[4];

src/panel/WaterfallPanel.h

@@ -0,0 +1,25 @@
+#pragma once
+
+#include "GLPanel.h"
+
+class WaterfallPanel : public GLPanel {
+public:
+    WaterfallPanel();
+    void setup(int fft_size_in, int num_waterfall_lines_in);
+    void refreshTheme();
+    void setPoints(std::vector<float> &points);
+    void step();
+    void drawPanelContents();
+    
+    std::vector<float> points;
+    bool needsUpdate;
+
+private:
+    GLuint waterfall[2];
+    int waterfall_ofs[2];
+    int fft_size;
+    int waterfall_lines;
+    unsigned char *waterfall_slice;
+    
+    ColorTheme *activeTheme;
+};

src/ui/GLPanel.cpp

@@ -13,6 +13,8 @@ GLPanel::GLPanel() : fillType(GLPANEL_FILL_SOLID), contentsVisible(true), transf
     fill[1] = RGB3f(0.1,0.1,0.1);
     borderColor = RGB3f(0.8, 0.8, 0.8);
     setCoordinateSystem(GLPANEL_Y_UP);
+    setMarginPx(0);
+    setBorderPx(0);
 }
 
 void GLPanel::genArrays() {

src/visual/WaterfallCanvas.cpp

@@ -35,10 +35,10 @@ EVT_MOUSEWHEEL(WaterfallCanvas::OnMouseWheelMoved)
 wxEND_EVENT_TABLE()
 
 WaterfallCanvas::WaterfallCanvas(wxWindow *parent, int *attribList) :
-        InteractiveCanvas(parent, attribList), dragState(WF_DRAG_NONE), nextDragState(WF_DRAG_NONE), fft_size(0), waterfall_lines(
+        InteractiveCanvas(parent, attribList), dragState(WF_DRAG_NONE), nextDragState(WF_DRAG_NONE), fft_size(0), waterfall_lines(0),
-                0), mouseZoom(1), zoom(1), hoverAlpha(1.0), dragOfs(0) {
+        dragOfs(0), mouseZoom(1), zoom(1), hoverAlpha(1.0) {
 
-    glContext = new WaterfallContext(this, &wxGetApp().GetContext(this));
+    glContext = new PrimaryGLContext(this, &wxGetApp().GetContext(this));
 
     SetCursor(wxCURSOR_CROSS);
 }
@@ -53,7 +53,7 @@ void WaterfallCanvas::setup(int fft_size_in, int waterfall_lines_in) {
     fft_size = fft_size_in;
     waterfall_lines = waterfall_lines_in;
 
-    glContext->Setup(fft_size, waterfall_lines);
+    waterfallPanel.setup(fft_size, waterfall_lines);
 }
 
 WaterfallCanvas::DragState WaterfallCanvas::getDragState() {
@@ -136,29 +136,27 @@ void WaterfallCanvas::OnPaint(wxPaintEvent& WXUNUSED(event)) {
         }
     }
     
-    bool step = false;
+    glContext->SetCurrent(*this);
+    initGLExtensions();
+    glViewport(0, 0, ClientSize.x, ClientSize.y);
+
     if (!visualDataQueue.empty()) {
         SpectrumVisualData *vData;
         
         visualDataQueue.pop(vData);
         
-        if (!vData) {
+        if (vData) {
-            return;
+            waterfallPanel.setPoints(vData->spectrum_points);
+            waterfallPanel.step();
         }
         
-        spectrum_points.assign(vData->spectrum_points.begin(),vData->spectrum_points.end());
-        
         vData->decRefCount();
-        step = true;
     }
 
-    
-    glContext->SetCurrent(*this);
-    initGLExtensions();
-    glViewport(0, 0, ClientSize.x, ClientSize.y);
-
     glContext->BeginDraw(0,0,0);
-    glContext->Draw(spectrum_points, step);
+
+    waterfallPanel.calcTransform(CubicVR::mat4::identity());
+    waterfallPanel.draw();
 
     std::vector<DemodulatorInstance *> &demods = wxGetApp().getDemodMgr().getDemodulators();
 

src/visual/WaterfallCanvas.h

@@ -7,9 +7,9 @@
 #include <queue>
 
 #include "InteractiveCanvas.h"
-#include "WaterfallContext.h"
 #include "MouseTracker.h"
 #include "SpectrumCanvas.h"
+#include "WaterfallPanel.h"
 
 
 class WaterfallCanvas: public InteractiveCanvas {
@@ -47,8 +47,8 @@ private:
     std::vector<float> spectrum_points;
 
     SpectrumCanvas *spectrumCanvas;
-
+    PrimaryGLContext *glContext;
-    WaterfallContext *glContext;
+    WaterfallPanel waterfallPanel;
 
     DragState dragState;
     DragState nextDragState;

src/visual/WaterfallContext.h

@@ -1,25 +0,0 @@
-#pragma once
-
-#include "PrimaryGLContext.h"
-#include "Gradient.h"
-#include "ColorTheme.h"
-
-class WaterfallCanvas;
-
-class WaterfallContext: public PrimaryGLContext {
-public:
-    WaterfallContext(WaterfallCanvas *canvas, wxGLContext *sharedContext);
-
-    void Draw(std::vector<float> &points, bool step);
-    void Setup(int fft_size_in, int num_waterfall_lines_in);
-    void refreshTheme();
-
-private:
-    GLuint waterfall[2];
-    int waterfall_ofs[2];
-    int fft_size;
-    int waterfall_lines;
-	unsigned char *waterfall_slice;
-
-    ColorTheme *activeTheme;
-};