From 3fd809888184c0e629bea2b628e2caca622bab88 Mon Sep 17 00:00:00 2001
From: Simon Ser <contact@emersion.fr>
Date: Tue, 15 Dec 2020 14:55:18 +0100
Subject: render/gles2: require GL_EXT_unpack_subimage

We implicitly depended on this extension.
---
 render/gles2/renderer.c | 5 +++++
 render/gles2/texture.c  | 1 -
 2 files changed, 5 insertions(+), 1 deletion(-)

(limited to 'render/gles2')

diff --git a/render/gles2/renderer.c b/render/gles2/renderer.c
index db313034..76114278 100644
--- a/render/gles2/renderer.c
+++ b/render/gles2/renderer.c
@@ -893,6 +893,11 @@ struct wlr_renderer *wlr_gles2_renderer_create(struct wlr_egl *egl) {
 		free(renderer);
 		return NULL;
 	}
+	if (!check_gl_ext(exts_str, "GL_EXT_unpack_subimage")) {
+		wlr_log(WLR_ERROR, "GL_EXT_unpack_subimage not supported");
+		free(renderer);
+		return NULL;
+	}
 
 	renderer->exts.read_format_bgra_ext =
 		check_gl_ext(exts_str, "GL_EXT_read_format_bgra");
diff --git a/render/gles2/texture.c b/render/gles2/texture.c
index 179bfd89..ea3a1161 100644
--- a/render/gles2/texture.c
+++ b/render/gles2/texture.c
@@ -55,7 +55,6 @@ static bool gles2_texture_write_pixels(struct wlr_texture *wlr_texture,
 		get_gles2_format_from_wl(texture->wl_format);
 	assert(fmt);
 
-	// TODO: what if the unpack subimage extension isn't supported?
 	push_gles2_debug(texture->renderer);
 
 	glBindTexture(GL_TEXTURE_2D, texture->tex);
-- 
cgit v1.2.3