wlr_drm_format: Don't take double pointer in wlr_drm_format_add

include/render/drm_format_set.h

@@ -5,7 +5,7 @@
 
 struct wlr_drm_format *wlr_drm_format_create(uint32_t format);
 bool wlr_drm_format_has(const struct wlr_drm_format *fmt, uint64_t modifier);
-bool wlr_drm_format_add(struct wlr_drm_format **fmt_ptr, uint64_t modifier);
+bool wlr_drm_format_add(struct wlr_drm_format *fmt, uint64_t modifier);
 struct wlr_drm_format *wlr_drm_format_dup(const struct wlr_drm_format *format);
 /**
  * Intersect modifiers for two DRM formats.

render/drm_format_set.c

@@ -61,14 +61,14 @@ bool wlr_drm_format_set_add(struct wlr_drm_format_set *set, uint32_t format,
 
 	struct wlr_drm_format **ptr = format_set_get_ref(set, format);
 	if (ptr) {
-		return wlr_drm_format_add(ptr, modifier);
+		return wlr_drm_format_add(*ptr, modifier);
 	}
 
 	struct wlr_drm_format *fmt = wlr_drm_format_create(format);
 	if (!fmt) {
 		return false;
 	}
-	if (!wlr_drm_format_add(&fmt, modifier)) {
+	if (!wlr_drm_format_add(fmt, modifier)) {
 		wlr_drm_format_finish(fmt);
 		return false;
 	}
@@ -114,9 +114,7 @@ bool wlr_drm_format_has(const struct wlr_drm_format *fmt, uint64_t modifier) {
 	return false;
 }
 
-bool wlr_drm_format_add(struct wlr_drm_format **fmt_ptr, uint64_t modifier) {
-	struct wlr_drm_format *fmt = *fmt_ptr;
-
+bool wlr_drm_format_add(struct wlr_drm_format *fmt, uint64_t modifier) {
 	if (wlr_drm_format_has(fmt, modifier)) {
 		return true;
 	}

types/output/swapchain.c

@@ -38,7 +38,7 @@ static struct wlr_swapchain *create_swapchain(struct wlr_output *output,
 		}
 
 		format->len = 0;
-		wlr_drm_format_add(&format, DRM_FORMAT_MOD_INVALID);
+		wlr_drm_format_add(format, DRM_FORMAT_MOD_INVALID);
 	}
 
 	struct wlr_swapchain *swapchain = wlr_swapchain_create(allocator, width, height, format);