GtkGraphView: Fix dispose function

* Chain to parent class's dispose function
* Move OpenGL frees to 'unrelaize' signal because the GL context is still available there.

.gitmodules

@@ -0,0 +1,3 @@
+[submodule "shimatta-opengl"]
+	path = shimatta-opengl
+	url = ssh://git@git.shimatta.de/mhu/shimatta-opengl.git

app/CMakeLists.txt

@@ -5,5 +5,5 @@ aux_source_directory("src" SRC_DIR)
 
 add_executable(${PROJECT_NAME} ${SRC_DIR})
 target_include_directories(${PROJECT_NAME} PRIVATE ${CMAKE_CURRENT_SOURCE_DIR}/include)
-target_link_libraries(${PROJECT_NAME} gtkgraphview)
+target_link_libraries(${PROJECT_NAME} gtkgraphview shimatta-opengl)
 add_dependencies(${PROJECT_NAME} gtkgraphview)

app/src/main.c

@@ -17,6 +17,7 @@
 #include <stdio.h>
 #include <gtk/gtk.h>
 #include <gtk-graph-view.h>
+#include <shimatta-opengl-program.h>
 
 static gboolean main_window_delete_event(GtkWidget *window, gpointer user)
 {

lib/CMakeLists.txt

@@ -9,9 +9,26 @@ pkg_check_modules(EPOXY REQUIRED epoxy)
 add_compile_options(-Wall -Wextra -Wold-style-declaration -Wuninitialized -Wmaybe-uninitialized -Wunused-parameter)
 aux_source_directory("src" SRC_DIR)
 
+IF(CMAKE_BUILD_TYPE STREQUAL "Debug")
+        message("${Yellow}Debug mode for shader locations used!${ColorReset}")
+	add_definitions(-DSHADER_DIRECTORY=\"${CMAKE_CURRENT_BINARY_DIR}/shaders\")
+	message("${BoldMagenta}${CMAKE_CURRENT_BINARY_DIR}/shaders used as shader directory${ColorReset}")
+else(CMAKE_BUILD_TYPE STREQUAL "Debug")
+	message("Global shader directory used. Make sure files in /usr/share/gtk-graph-view/shaders are available")
+	add_definitions(-DSHADER_DIRECTORY=\"/usr/share/gtk-graph-view/shaders\")
+ENDIF(CMAKE_BUILD_TYPE STREQUAL "Debug")
+
+
 add_library(${PROJECT_NAME} SHARED ${SRC_DIR})
 target_include_directories(${PROJECT_NAME} PUBLIC ${CMAKE_CURRENT_SOURCE_DIR}/include)
 target_include_directories(${PROJECT_NAME} PRIVATE ${EPOXY_INCLUDE_DIRS})
 target_link_libraries(${PROJECT_NAME} PUBLIC ${GLIB_LDFLAGS} ${GTK3_LDFLAGS} ${EPOXY_LDFLAGS} shimatta-opengl)
 target_link_directories(${PROJECT_NAME} PUBLIC ${GLIB_LINK_DIRS} ${GTK3_LINK_DIRS} ${EPOXY_LINK_DIRS})
 target_include_directories(${PROJECT_NAME} PUBLIC ${GLIB_INCLUDE_DIRS} ${GTK3_INCLUDE_DIRS})
+
+configure_file("${CMAKE_CURRENT_SOURCE_DIR}/shaders/passthrough3d.vertex.glsl"
+	"${CMAKE_CURRENT_BINARY_DIR}/shaders/passthrough3d.vertex.glsl" COPYONLY)
+configure_file("${CMAKE_CURRENT_SOURCE_DIR}/shaders/monochrome.fragment.glsl"
+	"${CMAKE_CURRENT_BINARY_DIR}/shaders/monochrome.fragment.glsl" COPYONLY)
+
+

lib/include/gtk-graph-view.h

@@ -1,7 +1,7 @@
 /*
  * This file is part of GtkGraphView.
  *
- * Foobar is free software: you can redistribute it and/or modify
+ * GtkGraphView is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 2 as
  * published by the Free Software Foundation.
  *

lib/shaders/monochrome.fragment.glsl

@@ -0,0 +1,10 @@
+#version 330 core
+
+out vec4 fragmentColor;
+
+uniform vec3 color;
+
+void main()
+{
+	fragmentColor = vec4(color.r, color.g, color.b, 1.0);
+}

lib/shaders/passthrough2d.vertex.glsl

@@ -0,0 +1,8 @@
+#version 330 core
+
+layout (location = 0) in vec2 inputVertex;
+
+void main()
+{
+	gl_Position = vec4(inputVertex.x, inputVertex.y, 0.0, 1.0);
+}

lib/shaders/passthrough3d.vertex.glsl

@@ -0,0 +1,8 @@
+#version 330 core
+
+layout (location = 0) in vec3 inputVertex;
+
+void main()
+{
+	gl_Position = vec4(inputVertex.x, inputVertex.y, inputVertex.z, 1.0);
+}

lib/src/gtk-graph-view.c

@@ -1,7 +1,7 @@
 /*
  * This file is part of GtkGraphView.
  *
-* Foobar is free software: you can redistribute it and/or modify
+* GtkGraphView is free software: you can redistribute it and/or modify
 * it under the terms of the GNU General Public License version 2 as
 * published by the Free Software Foundation.
 *
@@ -16,6 +16,12 @@
 
 #include <gtk-graph-view.h>
 #include <epoxy/gl.h>
+#include <shimatta-opengl-program.h>
+#include <shimatta-opengl-colored-triangle.h>
+
+#ifndef SHADER_DIRECTORY
+#define SHADER_DIRECTORY "/usr/share/gtk-graph-view/shaders"
+#endif
 
 struct _GtkGraphView {
 	GtkBox super;
@@ -24,10 +30,13 @@ struct _GtkGraphView {
 typedef struct {
 	/**
 	 * @brief The gl area displaying the graph view.
-	 * @note This variable is not referenced, because the gl_area is tightly bound to the GrapgView widget itself.
+	 * @note This variable is not referenced, because the gl_area is tightly bound to the GraphView widget itself.
 	 */
 	GtkWidget *gl_area;
 	GdkRGBA background_color;
+	ShimattaOpenglProgram *passthrough_prog;
+	ShimattaOpenglGraphics *test;
+
 } GtkGraphViewPrivate;
 
 G_DEFINE_TYPE_WITH_PRIVATE(GtkGraphView, gtk_graph_view, GTK_TYPE_BOX)
@@ -58,6 +67,9 @@ static gboolean gl_render(GtkGLArea *gl_area, GdkGLContext *context, GtkGraphVie
 
 	gl_area_set_clear_color(gl_area, &priv->background_color);
 	glClear(GL_COLOR_BUFFER_BIT);
+
+	shimatta_opengl_graphics_draw(priv->test);
+
 	glFlush();
 
 	return TRUE;
@@ -90,6 +102,14 @@ static void gtk_graph_view_set_property(GObject *obj, guint property_id, const G
 	}
 }
 
+void gtk_graph_view_dispose(GObject *obj)
+{
+	GtkGraphView *graph_view = GTK_GRAPH_VIEW(obj);
+	GtkGraphViewPrivate *priv = gtk_graph_view_get_instance_private(graph_view);
+
+	G_OBJECT_CLASS(gtk_graph_view_parent_class)->dispose(obj);
+}
+
 static GParamSpec *gtk_graph_view_properties[N_PROPERTIES] = {NULL};
 
 /**
@@ -103,6 +123,7 @@ static void gtk_graph_view_class_init(GtkGraphViewClass *klass)
 	oclass = G_OBJECT_CLASS(klass);
 	oclass->set_property = gtk_graph_view_set_property;
 	oclass->get_property = gtk_graph_view_get_property;
+	oclass->dispose = gtk_graph_view_dispose;
 
 	gtk_graph_view_properties[PROP_BG_COLOR] =
 			g_param_spec_boxed("background-color", "background color", "Background color of Graph View",
@@ -111,6 +132,50 @@ static void gtk_graph_view_class_init(GtkGraphViewClass *klass)
 	g_object_class_install_properties(oclass, N_PROPERTIES, gtk_graph_view_properties);
 }
 
+static void gl_area_realize(GtkGLArea *area, GtkGraphView *graph_view)
+{
+	GtkGraphViewPrivate *priv;
+	char error_text[512];
+	int status;
+	const vec3 color = {1.0f, 0.0f, 0.0f};
+	const vec3 vertecies[3] = {
+		{-0.5f, 0.0f, 0.0f},
+		{0.5f, 0.0f, 0.0f},
+		{0.0, 0.5f, 0.0f},
+	};
+
+	gtk_gl_area_make_current(area);
+	priv =	gtk_graph_view_get_instance_private(graph_view);
+
+	priv->passthrough_prog = shimatta_opengl_program_new_from_file(SHADER_DIRECTORY"/passthrough3d.vertex.glsl",
+								       NULL,
+								       SHADER_DIRECTORY"/monochrome.fragment.glsl");
+	status = shimatta_opengl_program_compile(priv->passthrough_prog, error_text, sizeof(error_text));
+	if (status) {
+		g_warning("Error compiling shader: %s", error_text);
+	}
+
+	priv->test = shimatta_opengl_colored_triangle_new(priv->passthrough_prog, color, vertecies);
+	shimatta_opengl_graphics_realize(priv->test);
+}
+
+static void gl_area_unrealize(GtkGLArea *area, GtkGraphView *graph_view)
+{
+	GtkGraphViewPrivate *priv;
+
+	gtk_gl_area_make_current(area);
+	priv = gtk_graph_view_get_instance_private(graph_view);
+
+	g_clear_object(&priv->passthrough_prog);
+}
+
+static void gl_area_resize(GtkGLArea *area, GtkGraphView *graph_view)
+{
+	(void)graph_view;
+
+	gtk_gl_area_queue_render(area);
+}
+
 /**
  * @brief Init a new GtkGraphView GObject
  * @param obj Object to initialize
@@ -124,6 +189,9 @@ static void gtk_graph_view_init(GtkGraphView *obj)
 	/* Create new GTK GL_Area as only child of the box */
 	priv->gl_area = gtk_gl_area_new();
 	g_signal_connect(priv->gl_area, "render", G_CALLBACK(gl_render), obj);
+	g_signal_connect(priv->gl_area, "realize", G_CALLBACK(gl_area_realize), obj);
+	g_signal_connect(priv->gl_area, "unrealize", G_CALLBACK(gl_area_unrealize), obj);
+	g_signal_connect(priv->gl_area, "resize", G_CALLBACK(gl_area_resize), obj);
 	gtk_container_add(GTK_CONTAINER(obj), priv->gl_area);
 	gtk_box_set_child_packing(GTK_BOX(obj), priv->gl_area, TRUE, TRUE, 0, GTK_PACK_START);
 	gtk_widget_show(priv->gl_area);

shimatta-opengl/CMakeLists.txt

@@ -1,17 +0,0 @@
-cmake_minimum_required(VERSION 3.5)
-project(shimatta-opengl LANGUAGES C)
-
-find_package(PkgConfig REQUIRED)
-pkg_search_module(GLIB REQUIRED glib-2.0)
-pkg_check_modules(EPOXY REQUIRED epoxy)
-
-add_compile_options(-Wall -Wextra -Wold-style-declaration -Wuninitialized -Wmaybe-uninitialized -Wunused-parameter)
-aux_source_directory("src" SRC_DIR)
-
-add_library(${PROJECT_NAME} SHARED ${SRC_DIR})
-target_include_directories(${PROJECT_NAME} PUBLIC ${CMAKE_CURRENT_SOURCE_DIR}/include)
-target_include_directories(${PROJECT_NAME} PRIVATE ${EPOXY_INCLUDE_DIRS})
-target_link_libraries(${PROJECT_NAME} PUBLIC ${GLIB_LDFLAGS} ${EPOXY_LDFLAGS})
-target_link_directories(${PROJECT_NAME} PUBLIC ${GLIB_LINK_DIRS} ${EPOXY_LINK_DIRS})
-target_include_directories(${PROJECT_NAME} PUBLIC ${GLIB_INCLUDE_DIRS})
-

shimatta-opengl/include/shimatta-opengl-program.h

@@ -1,75 +0,0 @@
-/*
- * This file is part of Shimatta OpenGL.
- *
-* Foobar is free software: you can redistribute it and/or modify
-* it under the terms of the GNU General Public License version 2 as
-* published by the Free Software Foundation.
-*
-* Shimatta OpenGL is distributed in the hope that it will be useful,
-* but WITHOUT ANY WARRANTY; without even the implied warranty of
-* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
-* GNU General Public License for more details.
-*
-* You should have received a copy of the GNU General Public License
-* along with GtkGraphView. If not, see <https://www.gnu.org/licenses/>.
-*/
-
-#include <glib.h>
-#include <glib-object.h>
-
-G_BEGIN_DECLS
-
-#define SHIMATTA_TYPE_OPENGL_PROGRAM (shimatta_opengl_program_get_type())
-
-G_DECLARE_FINAL_TYPE(ShimattaOpenglProgram, shimatta_opengl_program, SHIMATTA, OPENGL_PROGRAM, GObject);
-
-/**
- * @brief generate a new shader program using the source code for the shaders
- * @param vertex_shader Vertex shader source code
- * @param geometry_shader Geometry shader source code
- * @param fragment_shader Fragement shader source code
- * @return New shader program
- */
-ShimattaOpenglProgram *shimatta_opengl_program_new_from_data(const char *vertex_shader,
-                                                                const char *geometry_shader,
-                                                                const char *fragment_shader);
-
-/**
- * @brief shimatta_opengl_program_new_from_file
- * @param vertex_shader Source file for the vertex shader
- * @param geometry_shader Source file for the geometry shader
- * @param fragment_shader source file for the fragment shader
- * @return New shader program
- * @note The files have tobe accessible when calling @ref shimatta_opengl_program_compile
- */
-ShimattaOpenglProgram *shimatta_opengl_program_new_from_file(const char *vertex_shader,
-                                                const char *geometry_shader,
-                                                const char *fragment_shader);
-
-/**
- * @brief Compile the OpenGL Program
- * @param program Shader Program
- * @param error_text Error message from compilation. May be NULL.
- * @param error_text_size Size in bytes of the error_text buffer. May be 0.
- * @return 0 if successful. If Error error text will hold the error
- */
-int shimatta_opengl_program_compile(ShimattaOpenglProgram *program, char *error_text, size_t error_text_size);
-
-/**
- * @brief Active shader program.
- *
- * The activation only works, if the program is compiled successfully
- *
- * @param program Shader program
- * @return 0 if successful
- */
-int shimatta_opengl_program_use(ShimattaOpenglProgram *program);
-
-/**
- * @brief Check if shader program is compiled
- * @param program Shader program
- * @return true if successfully built
- */
-gboolean shimatta_opengl_program_is_compiled(ShimattaOpenglProgram *program);
-
-G_END_DECLS

shimatta-opengl/src/shimatta-opengl-program.c

@@ -1,75 +0,0 @@
-/*
- * This file is part of Shimatta OpenGL.
- *
-* Foobar is free software: you can redistribute it and/or modify
-* it under the terms of the GNU General Public License version 2 as
-* published by the Free Software Foundation.
-*
-* Shimatta OpenGL is distributed in the hope that it will be useful,
-* but WITHOUT ANY WARRANTY; without even the implied warranty of
-* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
-* GNU General Public License for more details.
-*
-* You should have received a copy of the GNU General Public License
-* along with GtkGraphView. If not, see <https://www.gnu.org/licenses/>.
-*/
-
-#include <shimatta-opengl-program.h>
-#include <epoxy/gl.h>
-
-struct _ShimattaOpenglProgram {
-	GObject super;
-};
-
-typedef struct {
-	GLint shader_id;
-	gboolean compiled;
-	char *fragment_file;
-	char *vertex_file;
-	char *geometry_file;
-	char *fragment_src;
-	char *vertex_src;
-	char *geometry_src;
-
-} ShimattaOpenglProgramPrivate;
-
-G_DEFINE_TYPE_WITH_PRIVATE(ShimattaOpenglProgram, shimatta_opengl_program, G_TYPE_OBJECT);
-
-static void shimatta_opengl_program_class_init(ShimattaOpenglProgramClass *klass)
-{
-	(void)klass;
-
-	return;
-}
-
-static void shimatta_opengl_program_init(ShimattaOpenglProgram *self)
-{
-	ShimattaOpenglProgramPrivate *priv;
-
-	priv = shimatta_opengl_program_get_instance_private(self);
-
-	priv->shader_id = 0;
-	priv->vertex_src = NULL;
-	priv->vertex_file = NULL;
-	priv->fragment_src = NULL;
-	priv->geometry_src = NULL;
-	priv->fragment_file = NULL;
-	priv->geometry_file = NULL;
-	priv->compiled = false;
-}
-
-ShimattaOpenglProgram *shimatta_opengl_program_new_from_data(const char *vertex_shader, const char *geometry_shader,
-								const char *fragment_shader)
-{
-	ShimattaOpenglProgram *ret;
-	ShimattaOpenglProgramPrivate *priv;
-
-	ret = g_object_new(SHIMATTA_TYPE_OPENGL_PROGRAM, NULL);
-	priv = shimatta_opengl_program_get_instance_private(ret);
-
-	priv->fragment_src = g_strdup(fragment_shader);
-	priv->geometry_src = g_strdup(geometry_shader);
-	priv->vertex_src= g_strdup(vertex_shader);
-
-	return ret;
-}