glvideomixer: don't clear pad->geometry_change when setting an identical value
authorMichiel Konstapel <github@konstapel.nl>
Mon, 16 Jan 2023 10:43:37 +0000 (11:43 +0100)
committerGStreamer Marge Bot <gitlab-merge-bot@gstreamer-foundation.org>
Wed, 18 Jan 2023 14:05:37 +0000 (14:05 +0000)
Closes #1715

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3733>

subprojects/gst-plugins-base/ext/gl/gstglvideomixer.c

index 0751c89..3926c9b 100644 (file)
@@ -919,25 +919,25 @@ gst_gl_video_mixer_pad_set_property (GObject * object, guint prop_id,
   switch (prop_id) {
     case PROP_PAD_XPOS:{
       gint val = g_value_get_int (value);
-      pad->geometry_change = val != pad->xpos;
+      pad->geometry_change |= val != pad->xpos;
       pad->xpos = val;
       break;
     }
     case PROP_PAD_YPOS:{
       gint val = g_value_get_int (value);
-      pad->geometry_change = val != pad->ypos;
+      pad->geometry_change |= val != pad->ypos;
       pad->ypos = val;
       break;
     }
     case PROP_PAD_WIDTH:{
       gint val = g_value_get_int (value);
-      pad->geometry_change = val != pad->width;
+      pad->geometry_change |= val != pad->width;
       pad->width = val;
       break;
     }
     case PROP_PAD_HEIGHT:{
       gint val = g_value_get_int (value);
-      pad->geometry_change = val != pad->height;
+      pad->geometry_change |= val != pad->height;
       pad->height = val;
     }
       break;
@@ -976,25 +976,25 @@ gst_gl_video_mixer_pad_set_property (GObject * object, guint prop_id,
       break;
     case PROP_PAD_CROP_LEFT:{
       gint val = g_value_get_int (value);
-      pad->geometry_change = val != pad->crop_left;
+      pad->geometry_change |= val != pad->crop_left;
       pad->crop_left = val;
       break;
     }
     case PROP_PAD_CROP_RIGHT:{
       gint val = g_value_get_int (value);
-      pad->geometry_change = val != pad->crop_right;
+      pad->geometry_change |= val != pad->crop_right;
       pad->crop_right = val;
       break;
     }
     case PROP_PAD_CROP_TOP:{
       gint val = g_value_get_int (value);
-      pad->geometry_change = val != pad->crop_top;
+      pad->geometry_change |= val != pad->crop_top;
       pad->crop_top = val;
       break;
     }
     case PROP_PAD_CROP_BOTTOM:{
       gint val = g_value_get_int (value);
-      pad->geometry_change = val != pad->crop_bottom;
+      pad->geometry_change |= val != pad->crop_bottom;
       pad->crop_bottom = val;
       break;
     }