From e93b9bbc02e2791f3a35b6f077fcbb8514c28aed Mon Sep 17 00:00:00 2001
From: Jedidiah Barber <contact@jedbarber.id.au>
Date: Sun, 12 Jan 2025 01:14:58 +1300
Subject: Refactored draw/handle methods in Widgets hierarchy, improved docs,
 added a few minor method bindings here and there

---
 src/c_fl_slider.cpp | 107 ++++++++++++++++++++++++++++------------------------
 1 file changed, 58 insertions(+), 49 deletions(-)

(limited to 'src/c_fl_slider.cpp')

diff --git a/src/c_fl_slider.cpp b/src/c_fl_slider.cpp
index dc55c90..fcdd32d 100644
--- a/src/c_fl_slider.cpp
+++ b/src/c_fl_slider.cpp
@@ -6,80 +6,68 @@
 
 #include <FL/Fl_Slider.H>
 #include "c_fl_slider.h"
-#include "c_fl_type.h"
 
 
 
 
-class My_Slider : public Fl_Slider {
-    public:
-        using Fl_Slider::Fl_Slider;
-        friend void slider_set_draw_hook(SLIDER s, void * d);
-        friend void fl_slider_draw(SLIDER s);
-        friend void slider_set_handle_hook(SLIDER s, void * h);
-        friend int fl_slider_handle(SLIDER s, int e);
-    protected:
-        void draw();
-        void real_draw();
-        int handle(int e);
-        int real_handle(int e);
-        d_hook_p draw_hook;
-        h_hook_p handle_hook;
+//  Exports from Ada
+
+extern "C" void widget_draw_hook(void * ud);
+extern "C" int widget_handle_hook(void * ud, int e);
+
+
+
+
+//  Non-friend protected access
+
+class Friend_Slider : Fl_Slider {
+public:
+    //  Really only needed for the (int,int,int,int) versions
+    using Fl_Slider::draw;
+    using Fl_Slider::handle;
 };
 
-void My_Slider::draw() {
-    (*draw_hook)(this->user_data());
-}
 
-void My_Slider::real_draw() {
-    Fl_Slider::draw();
-}
 
-int My_Slider::handle(int e) {
-    return (*handle_hook)(this->user_data(), e);
-}
 
-int My_Slider::real_handle(int e) {
-    return Fl_Slider::handle(e);
-}
+//  Attaching all relevant hooks and friends
 
-void slider_set_draw_hook(SLIDER s, void * d) {
-    reinterpret_cast<My_Slider*>(s)->draw_hook = reinterpret_cast<d_hook_p>(d);
-}
+class My_Slider : public Fl_Slider {
+public:
+    using Fl_Slider::Fl_Slider;
 
-void fl_slider_draw(SLIDER s) {
-    reinterpret_cast<My_Slider*>(s)->real_draw();
-}
+    friend void fl_slider_draw(SLIDER s);
+    friend int fl_slider_handle(SLIDER s, int e);
 
-void slider_set_handle_hook(SLIDER s, void * h) {
-    reinterpret_cast<My_Slider*>(s)->handle_hook = reinterpret_cast<h_hook_p>(h);
+    void draw();
+    int handle(int e);
+};
+
+void My_Slider::draw() {
+    widget_draw_hook(this->user_data());
 }
 
-int fl_slider_handle(SLIDER s, int e) {
-    return reinterpret_cast<My_Slider*>(s)->real_handle(e);
+int My_Slider::handle(int e) {
+    return widget_handle_hook(this->user_data(), e);
 }
 
 
 
 
+//  Flattened C API
+
 SLIDER new_fl_slider(int x, int y, int w, int h, char* label) {
     My_Slider *s = new My_Slider(x, y, w, h, label);
     return s;
 }
 
-void free_fl_slider(SLIDER s) {
-    delete reinterpret_cast<My_Slider*>(s);
-}
-
-
-
-
-int fl_slider_get_type(SLIDER s) {
-    return reinterpret_cast<Fl_Slider*>(s)->type();
+SLIDER new_fl_slider2(unsigned char k, int x, int y, int w, int h, char * label) {
+    My_Slider *s = new My_Slider(k, x, y, w, h, label);
+    return s;
 }
 
-void fl_slider_set_type(SLIDER s, int t) {
-    reinterpret_cast<Fl_Slider*>(s)->type(t);
+void free_fl_slider(SLIDER s) {
+    delete reinterpret_cast<My_Slider*>(s);
 }
 
 
@@ -101,7 +89,7 @@ float fl_slider_get_slider_size(SLIDER s) {
     return reinterpret_cast<Fl_Slider*>(s)->slider_size();
 }
 
-void fl_slider_set_slider_size(SLIDER s, float t) {
+void fl_slider_set_slider_size(SLIDER s, double t) {
     reinterpret_cast<Fl_Slider*>(s)->slider_size(t);
 }
 
@@ -110,3 +98,24 @@ int fl_slider_scrollvalue(SLIDER s, int p, int z, int f, int t) {
 }
 
 
+
+
+void fl_slider_draw(SLIDER s) {
+    reinterpret_cast<My_Slider*>(s)->Fl_Slider::draw();
+}
+
+void fl_slider_draw2(SLIDER s, int x, int y, int w, int h) {
+    void (Fl_Slider::*mydraw)(int,int,int,int) = &Friend_Slider::draw;
+    (reinterpret_cast<Fl_Slider*>(s)->*mydraw)(x, y, w, h);
+}
+
+int fl_slider_handle(SLIDER s, int e) {
+    return reinterpret_cast<My_Slider*>(s)->Fl_Slider::handle(e);
+}
+
+int fl_slider_handle2(SLIDER s, int e, int x, int y, int w, int h) {
+    int (Fl_Slider::*myhandle)(int,int,int,int,int) = &Friend_Slider::handle;
+    return (reinterpret_cast<Fl_Slider*>(s)->*myhandle)(e, x, y, w, h);
+}
+
+
-- 
cgit