[vlc-commits] opengl: merge prgm into vlc_gl_renderer

Romain Vimont git at videolan.org
Thu Feb 13 23:13:05 CET 2020


vlc | branch: master | Romain Vimont <rom1v at videolabs.io> | Thu Jan 23 17:56:55 2020 +0100| [d932092301860ec1b1fe945b5e961242f95be964] | committer: Jean-Baptiste Kempf

opengl: merge prgm into vlc_gl_renderer

The struct prgm contains some locations of the program, the struct
vlc_gl_renderer contains some others.

There are no reason to keep two separate structures.

Signed-off-by: Jean-Baptiste Kempf <jb at videolan.org>

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=d932092301860ec1b1fe945b5e961242f95be964
---

 modules/video_output/opengl/renderer.h    |  23 ++++-
 modules/video_output/opengl/vout_helper.c | 157 ++++++++++++------------------
 2 files changed, 85 insertions(+), 95 deletions(-)

diff --git a/modules/video_output/opengl/renderer.h b/modules/video_output/opengl/renderer.h
index a7be668802..756467d5f2 100644
--- a/modules/video_output/opengl/renderer.h
+++ b/modules/video_output/opengl/renderer.h
@@ -55,15 +55,34 @@ struct vlc_gl_renderer
      * has no default precision qualifier for floating point types. */
     const char *glsl_precision_header;
 
-    /* The following is used and filled by the opengl_fragment_shader_init
-     * function. */
+    GLuint program_id;
+
+    struct {
+        GLfloat OrientationMatrix[16];
+        GLfloat ProjectionMatrix[16];
+        GLfloat ZoomMatrix[16];
+        GLfloat ViewMatrix[16];
+    } var;
+
     struct {
         GLint Texture[PICTURE_PLANE_MAX];
         GLint TexSize[PICTURE_PLANE_MAX]; /* for GL_TEXTURE_RECTANGLE */
         GLint ConvMatrix;
         GLint FillColor;
         GLint *pl_vars; /* for pl_sh_res */
+
+        GLint TransformMatrix;
+        GLint OrientationMatrix;
+        GLint ProjectionMatrix;
+        GLint ViewMatrix;
+        GLint ZoomMatrix;
     } uloc;
+
+    struct {
+        GLint MultiTexCoord[3];
+        GLint VertexPosition;
+    } aloc;
+
     bool yuv_color;
     GLfloat conv_matrix[16];
 
diff --git a/modules/video_output/opengl/vout_helper.c b/modules/video_output/opengl/vout_helper.c
index 1abac866f6..678378fdfb 100644
--- a/modules/video_output/opengl/vout_helper.c
+++ b/modules/video_output/opengl/vout_helper.c
@@ -46,31 +46,6 @@
 
 #define SPHERE_RADIUS 1.f
 
-struct prgm
-{
-    GLuint id;
-    struct vlc_gl_renderer *renderer;
-
-    struct {
-        GLfloat OrientationMatrix[16];
-        GLfloat ProjectionMatrix[16];
-        GLfloat ZoomMatrix[16];
-        GLfloat ViewMatrix[16];
-    } var;
-
-    struct { /* UniformLocation */
-        GLint TransformMatrix;
-        GLint OrientationMatrix;
-        GLint ProjectionMatrix;
-        GLint ViewMatrix;
-        GLint ZoomMatrix;
-    } uloc;
-    struct { /* AttribLocation */
-        GLint MultiTexCoord[3];
-        GLint VertexPosition;
-    } aloc;
-};
-
 struct vout_display_opengl_t {
 
     vlc_gl_t   *gl;
@@ -83,8 +58,6 @@ struct vout_display_opengl_t {
 
     GLuint     texture[PICTURE_PLANE_MAX];
 
-    struct prgm prgm;
-
     unsigned nb_indices;
     GLuint vertex_buffer_object;
     GLuint index_buffer_object;
@@ -107,6 +80,7 @@ struct vout_display_opengl_t {
     float f_z;    /* Position of the camera on the shpere radius vector */
     float f_sar;
 
+    struct vlc_gl_renderer *renderer;
     struct vlc_gl_sub_renderer *sub_renderer;
 };
 
@@ -153,23 +127,23 @@ static void getProjectionMatrix(float sar, float fovy, GLfloat matrix[static 16]
 }
 
 static void getViewpointMatrixes(vout_display_opengl_t *vgl,
-                                 video_projection_mode_t projection_mode,
-                                 struct prgm *prgm)
+                                 video_projection_mode_t projection_mode)
 {
+    struct vlc_gl_renderer *renderer = vgl->renderer;
     if (projection_mode == PROJECTION_MODE_EQUIRECTANGULAR
         || projection_mode == PROJECTION_MODE_CUBEMAP_LAYOUT_STANDARD)
     {
-        getProjectionMatrix(vgl->f_sar, vgl->f_fovy, prgm->var.ProjectionMatrix);
-        getZoomMatrix(vgl->f_z, prgm->var.ZoomMatrix);
+        getProjectionMatrix(vgl->f_sar, vgl->f_fovy, renderer->var.ProjectionMatrix);
+        getZoomMatrix(vgl->f_z, renderer->var.ZoomMatrix);
 
         /* vgl->vp has been reversed and is a world transform */
-        vlc_viewpoint_to_4x4(&vgl->vp, prgm->var.ViewMatrix);
+        vlc_viewpoint_to_4x4(&vgl->vp, renderer->var.ViewMatrix);
     }
     else
     {
-        memcpy(prgm->var.ProjectionMatrix, identity, sizeof(identity));
-        memcpy(prgm->var.ZoomMatrix, identity, sizeof(identity));
-        memcpy(prgm->var.ViewMatrix, identity, sizeof(identity));
+        memcpy(renderer->var.ProjectionMatrix, identity, sizeof(identity));
+        memcpy(renderer->var.ZoomMatrix, identity, sizeof(identity));
+        memcpy(renderer->var.ViewMatrix, identity, sizeof(identity));
     }
 
 }
@@ -289,9 +263,8 @@ static GLuint BuildVertexShader(const struct vlc_gl_renderer *renderer,
 }
 
 static int
-opengl_link_program(struct prgm *prgm)
+opengl_link_program(struct vlc_gl_renderer *renderer)
 {
-    struct vlc_gl_renderer *renderer = prgm->renderer;
     struct vlc_gl_interop *interop = renderer->interop;
     const opengl_vtable_t *vt = renderer->vt;
 
@@ -332,24 +305,24 @@ opengl_link_program(struct prgm *prgm)
         }
     }
 
-    prgm->id = vt->CreateProgram();
-    vt->AttachShader(prgm->id, fragment_shader);
-    vt->AttachShader(prgm->id, vertex_shader);
-    vt->LinkProgram(prgm->id);
+    GLuint program_id = renderer->program_id = vt->CreateProgram();
+    vt->AttachShader(program_id, fragment_shader);
+    vt->AttachShader(program_id, vertex_shader);
+    vt->LinkProgram(program_id);
 
     vt->DeleteShader(vertex_shader);
     vt->DeleteShader(fragment_shader);
 
     /* Check program messages */
     int infoLength = 0;
-    vt->GetProgramiv(prgm->id, GL_INFO_LOG_LENGTH, &infoLength);
+    vt->GetProgramiv(program_id, GL_INFO_LOG_LENGTH, &infoLength);
     if (infoLength > 1)
     {
         char *infolog = malloc(infoLength);
         if (infolog != NULL)
         {
             int charsWritten;
-            vt->GetProgramInfoLog(prgm->id, infoLength, &charsWritten,
+            vt->GetProgramInfoLog(program_id, infoLength, &charsWritten,
                                   infolog);
             msg_Err(renderer->gl, "shader program: %s", infolog);
             free(infolog);
@@ -357,7 +330,7 @@ opengl_link_program(struct prgm *prgm)
 
         /* If there is some message, better to check linking is ok */
         GLint link_status = GL_TRUE;
-        vt->GetProgramiv(prgm->id, GL_LINK_STATUS, &link_status);
+        vt->GetProgramiv(program_id, GL_LINK_STATUS, &link_status);
         if (link_status == GL_FALSE)
         {
             msg_Err(renderer->gl, "Unable to use program");
@@ -367,15 +340,15 @@ opengl_link_program(struct prgm *prgm)
 
     /* Fetch UniformLocations and AttribLocations */
 #define GET_LOC(type, x, str) do { \
-    x = vt->Get##type##Location(prgm->id, str); \
+    x = vt->Get##type##Location(program_id, str); \
     assert(x != -1); \
     if (x == -1) { \
         msg_Err(renderer->gl, "Unable to Get"#type"Location(%s)", str); \
         goto error; \
     } \
 } while (0)
-#define GET_ULOC(x, str) GET_LOC(Uniform, prgm->uloc.x, str)
-#define GET_ALOC(x, str) GET_LOC(Attrib, prgm->aloc.x, str)
+#define GET_ULOC(x, str) GET_LOC(Uniform, renderer->uloc.x, str)
+#define GET_ALOC(x, str) GET_LOC(Attrib, renderer->aloc.x, str)
     GET_ULOC(TransformMatrix, "TransformMatrix");
     GET_ULOC(OrientationMatrix, "OrientationMatrix");
     GET_ULOC(ProjectionMatrix, "ProjectionMatrix");
@@ -388,15 +361,15 @@ opengl_link_program(struct prgm *prgm)
     if (interop->tex_count > 1)
         GET_ALOC(MultiTexCoord[1], "MultiTexCoord1");
     else
-        prgm->aloc.MultiTexCoord[1] = -1;
+        renderer->aloc.MultiTexCoord[1] = -1;
     if (interop->tex_count > 2)
         GET_ALOC(MultiTexCoord[2], "MultiTexCoord2");
     else
-        prgm->aloc.MultiTexCoord[2] = -1;
+        renderer->aloc.MultiTexCoord[2] = -1;
 #undef GET_LOC
 #undef GET_ULOC
 #undef GET_ALOC
-    int ret = prgm->renderer->pf_fetch_locations(prgm->renderer, prgm->id);
+    int ret = renderer->pf_fetch_locations(renderer, program_id);
     assert(ret == VLC_SUCCESS);
     if (ret != VLC_SUCCESS)
     {
@@ -407,18 +380,18 @@ opengl_link_program(struct prgm *prgm)
     return VLC_SUCCESS;
 
 error:
-    vt->DeleteProgram(prgm->id);
-    prgm->id = 0;
+    vt->DeleteProgram(program_id);
+    renderer->program_id = 0;
     return VLC_EGENERIC;
 }
 
 static void
-opengl_deinit_program(vout_display_opengl_t *vgl, struct prgm *prgm)
+opengl_deinit_program(vout_display_opengl_t *vgl)
 {
-    struct vlc_gl_renderer *renderer = prgm->renderer;
+    struct vlc_gl_renderer *renderer = vgl->renderer;
     vlc_gl_interop_Delete(renderer->interop);
-    if (prgm->id != 0)
-        vgl->vt.DeleteProgram(prgm->id);
+    if (renderer->program_id != 0)
+        vgl->vt.DeleteProgram(renderer->program_id);
 
 #ifdef HAVE_LIBPLACEBO
     FREENULL(renderer->uloc.pl_vars);
@@ -431,8 +404,7 @@ opengl_deinit_program(vout_display_opengl_t *vgl, struct prgm *prgm)
 
 static int
 opengl_init_program(vout_display_opengl_t *vgl, vlc_video_context *context,
-                    struct prgm *prgm, const video_format_t *fmt,
-                    bool b_dump_shaders)
+                    const video_format_t *fmt, bool b_dump_shaders)
 {
     struct vlc_gl_renderer *renderer = calloc(1, sizeof(*renderer));
     if (!renderer)
@@ -473,18 +445,18 @@ opengl_init_program(vout_display_opengl_t *vgl, vlc_video_context *context,
     }
 #endif
 
-    prgm->renderer = renderer;
+    vgl->renderer = renderer;
 
-    int ret = opengl_link_program(prgm);
+    int ret = opengl_link_program(renderer);
     if (ret != VLC_SUCCESS)
     {
-        opengl_deinit_program(vgl, prgm);
+        opengl_deinit_program(vgl);
         return VLC_EGENERIC;
     }
 
     getOrientationTransformMatrix(interop->fmt.orientation,
-                                  prgm->var.OrientationMatrix);
-    getViewpointMatrixes(vgl, interop->fmt.projection_mode, prgm);
+                                  renderer->var.OrientationMatrix);
+    getViewpointMatrixes(vgl, interop->fmt.projection_mode);
 
     return VLC_SUCCESS;
 }
@@ -680,8 +652,7 @@ vout_display_opengl_t *vout_display_opengl_New(video_format_t *fmt,
     }
 
     GL_ASSERT_NOERROR();
-    int ret = opengl_init_program(vgl, context, &vgl->prgm, fmt,
-                                  b_dump_shaders);
+    int ret = opengl_init_program(vgl, context, fmt, b_dump_shaders);
     if (ret != VLC_SUCCESS)
     {
         msg_Warn(gl, "could not init tex converter for %4.4s",
@@ -693,7 +664,7 @@ vout_display_opengl_t *vout_display_opengl_New(video_format_t *fmt,
 
     GL_ASSERT_NOERROR();
 
-    const struct vlc_gl_interop *interop = vgl->prgm.renderer->interop;
+    const struct vlc_gl_interop *interop = vgl->renderer->interop;
     /* Update the fmt to main program one */
     vgl->fmt = interop->fmt;
     /* The orientation is handled by the orientation matrix */
@@ -762,13 +733,13 @@ void vout_display_opengl_Delete(vout_display_opengl_t *vgl)
     vgl->vt.Finish();
     vgl->vt.Flush();
 
-    const struct vlc_gl_interop *interop = vgl->prgm.renderer->interop;
+    const struct vlc_gl_interop *interop = vgl->renderer->interop;
     const size_t main_tex_count = interop->tex_count;
     const bool main_del_texs = !interop->handle_texs_gen;
 
     vlc_gl_sub_renderer_Delete(vgl->sub_renderer);
 
-    opengl_deinit_program(vgl, &vgl->prgm);
+    opengl_deinit_program(vgl);
 
     vgl->vt.DeleteBuffers(1, &vgl->vertex_buffer_object);
     vgl->vt.DeleteBuffers(1, &vgl->index_buffer_object);
@@ -831,7 +802,7 @@ int vout_display_opengl_SetViewpoint(vout_display_opengl_t *vgl,
         UpdateFOVy(vgl);
         UpdateZ(vgl);
     }
-    getViewpointMatrixes(vgl, vgl->fmt.projection_mode, &vgl->prgm);
+    getViewpointMatrixes(vgl, vgl->fmt.projection_mode);
 
     return VLC_SUCCESS;
 }
@@ -846,7 +817,7 @@ void vout_display_opengl_SetWindowAspectRatio(vout_display_opengl_t *vgl,
     vgl->f_sar = f_sar;
     UpdateFOVy(vgl);
     UpdateZ(vgl);
-    getViewpointMatrixes(vgl, vgl->fmt.projection_mode, &vgl->prgm);
+    getViewpointMatrixes(vgl, vgl->fmt.projection_mode);
 }
 
 void vout_display_opengl_Viewport(vout_display_opengl_t *vgl, int x, int y,
@@ -860,7 +831,7 @@ int vout_display_opengl_Prepare(vout_display_opengl_t *vgl,
 {
     GL_ASSERT_NOERROR();
 
-    struct vlc_gl_renderer *renderer = vgl->prgm.renderer;
+    struct vlc_gl_renderer *renderer = vgl->renderer;
     const struct vlc_gl_interop *interop = renderer->interop;
 
     /* Update the texture */
@@ -1146,7 +1117,7 @@ static int SetupCoords(vout_display_opengl_t *vgl,
                        const float *left, const float *top,
                        const float *right, const float *bottom)
 {
-    const struct vlc_gl_interop *interop = vgl->prgm.renderer->interop;
+    const struct vlc_gl_interop *interop = vgl->renderer->interop;
 
     GLfloat *vertexCoord, *textureCoord;
     GLushort *indices;
@@ -1207,9 +1178,9 @@ static int SetupCoords(vout_display_opengl_t *vgl,
     return VLC_SUCCESS;
 }
 
-static void DrawWithShaders(vout_display_opengl_t *vgl, struct prgm *prgm)
+static void DrawWithShaders(vout_display_opengl_t *vgl)
 {
-    struct vlc_gl_renderer *renderer = prgm->renderer;
+    struct vlc_gl_renderer *renderer = vgl->renderer;
     const struct vlc_gl_interop *interop = renderer->interop;
     renderer->pf_prepare_shader(renderer, vgl->tex_width, vgl->tex_height, 1.0f);
 
@@ -1220,16 +1191,16 @@ static void DrawWithShaders(vout_display_opengl_t *vgl, struct prgm *prgm)
 
         vgl->vt.BindBuffer(GL_ARRAY_BUFFER, vgl->texture_buffer_object[j]);
 
-        assert(prgm->aloc.MultiTexCoord[j] != -1);
-        vgl->vt.EnableVertexAttribArray(prgm->aloc.MultiTexCoord[j]);
-        vgl->vt.VertexAttribPointer(prgm->aloc.MultiTexCoord[j], 2, GL_FLOAT,
-                                     0, 0, 0);
+        assert(renderer->aloc.MultiTexCoord[j] != -1);
+        vgl->vt.EnableVertexAttribArray(renderer->aloc.MultiTexCoord[j]);
+        vgl->vt.VertexAttribPointer(renderer->aloc.MultiTexCoord[j], 2,
+                                    GL_FLOAT, 0, 0, 0);
     }
 
     vgl->vt.BindBuffer(GL_ARRAY_BUFFER, vgl->vertex_buffer_object);
     vgl->vt.BindBuffer(GL_ELEMENT_ARRAY_BUFFER, vgl->index_buffer_object);
-    vgl->vt.EnableVertexAttribArray(prgm->aloc.VertexPosition);
-    vgl->vt.VertexAttribPointer(prgm->aloc.VertexPosition, 3, GL_FLOAT, 0, 0, 0);
+    vgl->vt.EnableVertexAttribArray(renderer->aloc.VertexPosition);
+    vgl->vt.VertexAttribPointer(renderer->aloc.VertexPosition, 3, GL_FLOAT, 0, 0, 0);
 
     const GLfloat *tm = NULL;
     if (interop->ops && interop->ops->get_transform_matrix)
@@ -1237,16 +1208,16 @@ static void DrawWithShaders(vout_display_opengl_t *vgl, struct prgm *prgm)
     if (!tm)
         tm = identity;
 
-    vgl->vt.UniformMatrix4fv(prgm->uloc.TransformMatrix, 1, GL_FALSE, tm);
+    vgl->vt.UniformMatrix4fv(renderer->uloc.TransformMatrix, 1, GL_FALSE, tm);
 
-    vgl->vt.UniformMatrix4fv(prgm->uloc.OrientationMatrix, 1, GL_FALSE,
-                             prgm->var.OrientationMatrix);
-    vgl->vt.UniformMatrix4fv(prgm->uloc.ProjectionMatrix, 1, GL_FALSE,
-                             prgm->var.ProjectionMatrix);
-    vgl->vt.UniformMatrix4fv(prgm->uloc.ViewMatrix, 1, GL_FALSE,
-                             prgm->var.ViewMatrix);
-    vgl->vt.UniformMatrix4fv(prgm->uloc.ZoomMatrix, 1, GL_FALSE,
-                             prgm->var.ZoomMatrix);
+    vgl->vt.UniformMatrix4fv(renderer->uloc.OrientationMatrix, 1, GL_FALSE,
+                             renderer->var.OrientationMatrix);
+    vgl->vt.UniformMatrix4fv(renderer->uloc.ProjectionMatrix, 1, GL_FALSE,
+                             renderer->var.ProjectionMatrix);
+    vgl->vt.UniformMatrix4fv(renderer->uloc.ViewMatrix, 1, GL_FALSE,
+                             renderer->var.ViewMatrix);
+    vgl->vt.UniformMatrix4fv(renderer->uloc.ZoomMatrix, 1, GL_FALSE,
+                             renderer->var.ZoomMatrix);
 
     vgl->vt.DrawElements(GL_TRIANGLES, vgl->nb_indices, GL_UNSIGNED_SHORT, 0);
 }
@@ -1274,7 +1245,7 @@ static void TextureCropForStereo(vout_display_opengl_t *vgl,
                                  float *left, float *top,
                                  float *right, float *bottom)
 {
-    const struct vlc_gl_interop *interop = vgl->prgm.renderer->interop;
+    const struct vlc_gl_interop *interop = vgl->renderer->interop;
 
     float stereoCoefs[2];
     float stereoOffsets[2];
@@ -1312,7 +1283,7 @@ int vout_display_opengl_Display(vout_display_opengl_t *vgl,
        Currently, the OS X provider uses it to get a smooth window resizing */
     vgl->vt.Clear(GL_COLOR_BUFFER_BIT);
 
-    vgl->vt.UseProgram(vgl->prgm.id);
+    vgl->vt.UseProgram(vgl->renderer->program_id);
 
     if (source->i_x_offset != vgl->last_source.i_x_offset
      || source->i_y_offset != vgl->last_source.i_y_offset
@@ -1323,7 +1294,7 @@ int vout_display_opengl_Display(vout_display_opengl_t *vgl,
         float top[PICTURE_PLANE_MAX];
         float right[PICTURE_PLANE_MAX];
         float bottom[PICTURE_PLANE_MAX];
-        const struct vlc_gl_renderer *renderer = vgl->prgm.renderer;
+        const struct vlc_gl_renderer *renderer = vgl->renderer;
         const struct vlc_gl_interop *interop = renderer->interop;
         for (unsigned j = 0; j < interop->tex_count; j++)
         {
@@ -1359,7 +1330,7 @@ int vout_display_opengl_Display(vout_display_opengl_t *vgl,
         vgl->last_source.i_visible_width = source->i_visible_width;
         vgl->last_source.i_visible_height = source->i_visible_height;
     }
-    DrawWithShaders(vgl, &vgl->prgm);
+    DrawWithShaders(vgl);
 
     int ret = vlc_gl_sub_renderer_Draw(vgl->sub_renderer);
     if (ret != VLC_SUCCESS)



More information about the vlc-commits mailing list