react-native-webrtc-kaleidoscope 2.0.0 → 2.1.0

package/android/src/main/java/com/simiancraft/kaleidoscope/gpu/Egl.kt

@@ -9,80 +9,92 @@ package com.simiancraft.kaleidoscope.gpu
 import android.opengl.GLES30
 
 internal object Egl {
-  private const val GL_TEXTURE_BINDING_EXTERNAL_OES = 0x8D67
-  private const val GL_TEXTURE_EXTERNAL_OES = 0x8D65
+    private const val GL_TEXTURE_BINDING_EXTERNAL_OES = 0x8D67
+    private const val GL_TEXTURE_EXTERNAL_OES = 0x8D65
 
-  /**
-   * Convert an android.graphics.Matrix (3x3 row-major affine) into a flat 16-
-   * element column-major float array suitable for glUniformMatrix4fv. Z row
-   * and column become identity.
-   *
-   * android.graphics.Matrix values layout (from Matrix.getValues):
-   *   [m00 m01 m02]   [a b c]
-   *   [m10 m11 m12] = [d e f]
-   *   [m20 m21 m22]   [g h i]
-   *
-   * Caller is responsible for the float array's lifetime (no per-call alloc
-   * mitigation here; if it matters, reuse a buffer).
-   */
-  fun matrixToGl(matrix: android.graphics.Matrix): FloatArray {
-    val v = FloatArray(9)
-    matrix.getValues(v)
-    return floatArrayOf(
-      v[0], v[3], 0f, v[6],
-      v[1], v[4], 0f, v[7],
-      0f,   0f,   1f, 0f,
-      v[2], v[5], 0f, v[8],
-    )
-  }
-
-  data class State(
-    val viewport: IntArray,
-    val activeTexture: Int,
-    val texture2DBinding: Int,
-    val textureExternalBinding: Int,
-    val program: Int,
-    val arrayBuffer: Int,
-    val vao: Int,
-    val framebuffer: Int,
-    val blendEnabled: Boolean,
-    val depthTestEnabled: Boolean,
-  )
+    /**
+     * Convert an android.graphics.Matrix (3x3 row-major affine) into a flat 16-
+     * element column-major float array suitable for glUniformMatrix4fv. Z row
+     * and column become identity.
+     *
+     * android.graphics.Matrix values layout (from Matrix.getValues):
+     *   [m00 m01 m02]   [a b c]
+     *   [m10 m11 m12] = [d e f]
+     *   [m20 m21 m22]   [g h i]
+     *
+     * Caller is responsible for the float array's lifetime (no per-call alloc
+     * mitigation here; if it matters, reuse a buffer).
+     */
+    fun matrixToGl(matrix: android.graphics.Matrix): FloatArray {
+        val v = FloatArray(9)
+        matrix.getValues(v)
+        return floatArrayOf(
+            v[0],
+            v[3],
+            0f,
+            v[6],
+            v[1],
+            v[4],
+            0f,
+            v[7],
+            0f,
+            0f,
+            1f,
+            0f,
+            v[2],
+            v[5],
+            0f,
+            v[8],
+        )
+    }
 
-  fun save(): State =
-    State(
-      viewport = IntArray(4).also { GLES30.glGetIntegerv(GLES30.GL_VIEWPORT, it, 0) },
-      activeTexture = getInt(GLES30.GL_ACTIVE_TEXTURE),
-      texture2DBinding = getInt(GLES30.GL_TEXTURE_BINDING_2D),
-      textureExternalBinding = getInt(GL_TEXTURE_BINDING_EXTERNAL_OES),
-      program = getInt(GLES30.GL_CURRENT_PROGRAM),
-      arrayBuffer = getInt(GLES30.GL_ARRAY_BUFFER_BINDING),
-      vao = getInt(GLES30.GL_VERTEX_ARRAY_BINDING),
-      framebuffer = getInt(GLES30.GL_FRAMEBUFFER_BINDING),
-      blendEnabled = GLES30.glIsEnabled(GLES30.GL_BLEND),
-      depthTestEnabled = GLES30.glIsEnabled(GLES30.GL_DEPTH_TEST),
+    data class State(
+        val viewport: IntArray,
+        val activeTexture: Int,
+        val texture2DBinding: Int,
+        val textureExternalBinding: Int,
+        val program: Int,
+        val arrayBuffer: Int,
+        val vao: Int,
+        val framebuffer: Int,
+        val blendEnabled: Boolean,
+        val depthTestEnabled: Boolean,
     )
 
-  fun restore(s: State) {
-    GLES30.glViewport(s.viewport[0], s.viewport[1], s.viewport[2], s.viewport[3])
-    GLES30.glActiveTexture(s.activeTexture)
-    GLES30.glBindTexture(GLES30.GL_TEXTURE_2D, s.texture2DBinding)
-    GLES30.glBindTexture(GL_TEXTURE_EXTERNAL_OES, s.textureExternalBinding)
-    GLES30.glUseProgram(s.program)
-    GLES30.glBindBuffer(GLES30.GL_ARRAY_BUFFER, s.arrayBuffer)
-    GLES30.glBindVertexArray(s.vao)
-    GLES30.glBindFramebuffer(GLES30.GL_FRAMEBUFFER, s.framebuffer)
-    if (s.blendEnabled) GLES30.glEnable(GLES30.GL_BLEND) else GLES30.glDisable(GLES30.GL_BLEND)
-    if (s.depthTestEnabled) {
-      GLES30.glEnable(GLES30.GL_DEPTH_TEST)
-    } else {
-      GLES30.glDisable(GLES30.GL_DEPTH_TEST)
+    fun save(): State =
+        State(
+            viewport = IntArray(4).also { GLES30.glGetIntegerv(GLES30.GL_VIEWPORT, it, 0) },
+            activeTexture = getInt(GLES30.GL_ACTIVE_TEXTURE),
+            texture2DBinding = getInt(GLES30.GL_TEXTURE_BINDING_2D),
+            textureExternalBinding = getInt(GL_TEXTURE_BINDING_EXTERNAL_OES),
+            program = getInt(GLES30.GL_CURRENT_PROGRAM),
+            arrayBuffer = getInt(GLES30.GL_ARRAY_BUFFER_BINDING),
+            vao = getInt(GLES30.GL_VERTEX_ARRAY_BINDING),
+            framebuffer = getInt(GLES30.GL_FRAMEBUFFER_BINDING),
+            blendEnabled = GLES30.glIsEnabled(GLES30.GL_BLEND),
+            depthTestEnabled = GLES30.glIsEnabled(GLES30.GL_DEPTH_TEST),
+        )
+
+    fun restore(s: State) {
+        GLES30.glViewport(s.viewport[0], s.viewport[1], s.viewport[2], s.viewport[3])
+        GLES30.glActiveTexture(s.activeTexture)
+        GLES30.glBindTexture(GLES30.GL_TEXTURE_2D, s.texture2DBinding)
+        GLES30.glBindTexture(GL_TEXTURE_EXTERNAL_OES, s.textureExternalBinding)
+        GLES30.glUseProgram(s.program)
+        GLES30.glBindBuffer(GLES30.GL_ARRAY_BUFFER, s.arrayBuffer)
+        GLES30.glBindVertexArray(s.vao)
+        GLES30.glBindFramebuffer(GLES30.GL_FRAMEBUFFER, s.framebuffer)
+        if (s.blendEnabled) GLES30.glEnable(GLES30.GL_BLEND) else GLES30.glDisable(GLES30.GL_BLEND)
+        if (s.depthTestEnabled) {
+            GLES30.glEnable(GLES30.GL_DEPTH_TEST)
+        } else {
+            GLES30.glDisable(GLES30.GL_DEPTH_TEST)
+        }
     }
-  }
 
-  private fun getInt(pname: Int): Int {
-    val out = IntArray(1)
-    GLES30.glGetIntegerv(pname, out, 0)
-    return out[0]
-  }
+    private fun getInt(pname: Int): Int {
+        val out = IntArray(1)
+        GLES30.glGetIntegerv(pname, out, 0)
+        return out[0]
+    }
 }

package/android/src/main/java/com/simiancraft/kaleidoscope/gpu/Fbo.kt

@@ -6,56 +6,59 @@ package com.simiancraft.kaleidoscope.gpu
 
 import android.opengl.GLES30
 
-internal class Fbo(val width: Int, val height: Int) {
-  val texture: Int
-  val framebuffer: Int
+internal class Fbo(
+    val width: Int,
+    val height: Int,
+) {
+    val texture: Int
+    val framebuffer: Int
 
-  init {
-    val texIds = IntArray(1)
-    GLES30.glGenTextures(1, texIds, 0)
-    texture = texIds[0]
-    GLES30.glBindTexture(GLES30.GL_TEXTURE_2D, texture)
-    GLES30.glTexImage2D(
-      GLES30.GL_TEXTURE_2D,
-      0,
-      GLES30.GL_RGBA,
-      width,
-      height,
-      0,
-      GLES30.GL_RGBA,
-      GLES30.GL_UNSIGNED_BYTE,
-      null,
-    )
-    GLES30.glTexParameteri(GLES30.GL_TEXTURE_2D, GLES30.GL_TEXTURE_MIN_FILTER, GLES30.GL_LINEAR)
-    GLES30.glTexParameteri(GLES30.GL_TEXTURE_2D, GLES30.GL_TEXTURE_MAG_FILTER, GLES30.GL_LINEAR)
-    GLES30.glTexParameteri(GLES30.GL_TEXTURE_2D, GLES30.GL_TEXTURE_WRAP_S, GLES30.GL_CLAMP_TO_EDGE)
-    GLES30.glTexParameteri(GLES30.GL_TEXTURE_2D, GLES30.GL_TEXTURE_WRAP_T, GLES30.GL_CLAMP_TO_EDGE)
+    init {
+        val texIds = IntArray(1)
+        GLES30.glGenTextures(1, texIds, 0)
+        texture = texIds[0]
+        GLES30.glBindTexture(GLES30.GL_TEXTURE_2D, texture)
+        GLES30.glTexImage2D(
+            GLES30.GL_TEXTURE_2D,
+            0,
+            GLES30.GL_RGBA,
+            width,
+            height,
+            0,
+            GLES30.GL_RGBA,
+            GLES30.GL_UNSIGNED_BYTE,
+            null,
+        )
+        GLES30.glTexParameteri(GLES30.GL_TEXTURE_2D, GLES30.GL_TEXTURE_MIN_FILTER, GLES30.GL_LINEAR)
+        GLES30.glTexParameteri(GLES30.GL_TEXTURE_2D, GLES30.GL_TEXTURE_MAG_FILTER, GLES30.GL_LINEAR)
+        GLES30.glTexParameteri(GLES30.GL_TEXTURE_2D, GLES30.GL_TEXTURE_WRAP_S, GLES30.GL_CLAMP_TO_EDGE)
+        GLES30.glTexParameteri(GLES30.GL_TEXTURE_2D, GLES30.GL_TEXTURE_WRAP_T, GLES30.GL_CLAMP_TO_EDGE)
 
-    val fboIds = IntArray(1)
-    GLES30.glGenFramebuffers(1, fboIds, 0)
-    framebuffer = fboIds[0]
-    GLES30.glBindFramebuffer(GLES30.GL_FRAMEBUFFER, framebuffer)
-    GLES30.glFramebufferTexture2D(
-      GLES30.GL_FRAMEBUFFER,
-      GLES30.GL_COLOR_ATTACHMENT0,
-      GLES30.GL_TEXTURE_2D,
-      texture,
-      0,
-    )
-    val status = GLES30.glCheckFramebufferStatus(GLES30.GL_FRAMEBUFFER)
-    if (status != GLES30.GL_FRAMEBUFFER_COMPLETE) {
-      error("Kaleidoscope: FBO incomplete: $status")
+        val fboIds = IntArray(1)
+        GLES30.glGenFramebuffers(1, fboIds, 0)
+        framebuffer = fboIds[0]
+        GLES30.glBindFramebuffer(GLES30.GL_FRAMEBUFFER, framebuffer)
+        GLES30.glFramebufferTexture2D(
+            GLES30.GL_FRAMEBUFFER,
+            GLES30.GL_COLOR_ATTACHMENT0,
+            GLES30.GL_TEXTURE_2D,
+            texture,
+            0,
+        )
+        val status = GLES30.glCheckFramebufferStatus(GLES30.GL_FRAMEBUFFER)
+        if (status != GLES30.GL_FRAMEBUFFER_COMPLETE) {
+            error("Kaleidoscope: FBO incomplete: $status")
+        }
+        GLES30.glBindFramebuffer(GLES30.GL_FRAMEBUFFER, 0)
     }
-    GLES30.glBindFramebuffer(GLES30.GL_FRAMEBUFFER, 0)
-  }
 
-  fun bind() {
-    GLES30.glBindFramebuffer(GLES30.GL_FRAMEBUFFER, framebuffer)
-    GLES30.glViewport(0, 0, width, height)
-  }
+    fun bind() {
+        GLES30.glBindFramebuffer(GLES30.GL_FRAMEBUFFER, framebuffer)
+        GLES30.glViewport(0, 0, width, height)
+    }
 
-  fun delete() {
-    GLES30.glDeleteFramebuffers(1, intArrayOf(framebuffer), 0)
-    GLES30.glDeleteTextures(1, intArrayOf(texture), 0)
-  }
+    fun delete() {
+        GLES30.glDeleteFramebuffers(1, intArrayOf(framebuffer), 0)
+        GLES30.glDeleteTextures(1, intArrayOf(texture), 0)
+    }
 }

package/android/src/main/java/com/simiancraft/kaleidoscope/gpu/FramePipeline.kt

@@ -38,121 +38,122 @@ import android.opengl.GLES30
 import android.util.Log
 
 internal class FramePipeline {
-  /**
-   * A rendered-but-not-yet-handed-off output texture and the fence that
-   * signals when its GPU work has completed. The caller has already detached
-   * and freed the FBO; only the color texture survives here.
-   */
-  private class PendingFrame(
-    val textureId: Int,
-    val width: Int,
-    val height: Int,
-    val rotation: Int,
-    val timestampNs: Long,
-    val fence: Long,
-    // Wall-clock at the moment this frame's GPU work was submitted (the
-    // enqueue() call that fenced it). Used only when debug timing is on, to
-    // report GPU-completion latency a frame late without adding a blocking
-    // wait of its own.
-    val submitNanos: Long,
-  )
+    /**
+     * A rendered-but-not-yet-handed-off output texture and the fence that
+     * signals when its GPU work has completed. The caller has already detached
+     * and freed the FBO; only the color texture survives here.
+     */
+    private class PendingFrame(
+        val textureId: Int,
+        val width: Int,
+        val height: Int,
+        val rotation: Int,
+        val timestampNs: Long,
+        val fence: Long,
+        // Wall-clock at the moment this frame's GPU work was submitted (the
+        // enqueue() call that fenced it). Used only when debug timing is on, to
+        // report GPU-completion latency a frame late without adding a blocking
+        // wait of its own.
+        val submitNanos: Long,
+    )
 
-  private var pending: PendingFrame? = null
+    private var pending: PendingFrame? = null
 
-  /**
-   * Insert a sync fence into the GL command stream right after the caller's
-   * final draw, then enqueue the freshly rendered texture and return the
-   * PREVIOUS frame's texture (now GPU-complete) for handoff downstream.
-   *
-   * Must be called on the GL thread after the output FBO has been detached
-   * and deleted (so only `textureId` remains live).
-   *
-   * @return a [Ready] describing the previous frame to wrap in a
-   *   TextureBufferImpl, or null on the first frame (caller forwards original).
-   */
-  fun enqueue(
-    textureId: Int,
-    width: Int,
-    height: Int,
-    rotation: Int,
-    timestampNs: Long,
-    debugTiming: Boolean = false,
-    timingLabel: String = "",
-  ): Ready? {
-    // Fence the current frame's GPU work. glFlush guarantees the fence is
-    // actually in the command queue so a later glClientWaitSync without the
-    // flush-commands bit can still observe it.
-    val fence = GLES30.glFenceSync(GLES30.GL_SYNC_GPU_COMMANDS_COMPLETE, 0)
-    GLES30.glFlush()
+    /**
+     * Insert a sync fence into the GL command stream right after the caller's
+     * final draw, then enqueue the freshly rendered texture and return the
+     * PREVIOUS frame's texture (now GPU-complete) for handoff downstream.
+     *
+     * Must be called on the GL thread after the output FBO has been detached
+     * and deleted (so only `textureId` remains live).
+     *
+     * @return a [Ready] describing the previous frame to wrap in a
+     *   TextureBufferImpl, or null on the first frame (caller forwards original).
+     */
+    fun enqueue(
+        textureId: Int,
+        width: Int,
+        height: Int,
+        rotation: Int,
+        timestampNs: Long,
+        debugTiming: Boolean = false,
+        timingLabel: String = "",
+    ): Ready? {
+        // Fence the current frame's GPU work. glFlush guarantees the fence is
+        // actually in the command queue so a later glClientWaitSync without the
+        // flush-commands bit can still observe it.
+        val fence = GLES30.glFenceSync(GLES30.GL_SYNC_GPU_COMMANDS_COMPLETE, 0)
+        GLES30.glFlush()
 
-    val submitNanos = if (debugTiming) System.nanoTime() else 0L
-    val prev = pending
-    pending = PendingFrame(textureId, width, height, rotation, timestampNs, fence, submitNanos)
+        val submitNanos = if (debugTiming) System.nanoTime() else 0L
+        val prev = pending
+        pending = PendingFrame(textureId, width, height, rotation, timestampNs, fence, submitNanos)
 
-    if (prev == null) return null
+        if (prev == null) return null
 
-    // Wait on the PREVIOUS frame's fence. It has had a full frame interval to
-    // complete, so this is almost always GL_ALREADY_SIGNALED. The timeout is a
-    // backstop against a stalled GPU; we still hand the texture off afterward
-    // because the consumer's EGL context shares ours and would otherwise read
-    // a torn texture. Pass the flush-commands bit defensively in case the
-    // predecessor's flush has not yet been issued on this context.
-    val waitResult = GLES30.glClientWaitSync(
-      prev.fence,
-      GLES30.GL_SYNC_FLUSH_COMMANDS_BIT,
-      FENCE_TIMEOUT_NS,
-    )
-    if (waitResult == GLES30.GL_TIMEOUT_EXPIRED || waitResult == GLES30.GL_WAIT_FAILED) {
-      Log.w(TAG, "fence wait returned 0x${waitResult.toString(16)}; handing off anyway")
-    }
-    GLES30.glDeleteSync(prev.fence)
+        // Wait on the PREVIOUS frame's fence. It has had a full frame interval to
+        // complete, so this is almost always GL_ALREADY_SIGNALED. The timeout is a
+        // backstop against a stalled GPU; we still hand the texture off afterward
+        // because the consumer's EGL context shares ours and would otherwise read
+        // a torn texture. Pass the flush-commands bit defensively in case the
+        // predecessor's flush has not yet been issued on this context.
+        val waitResult =
+            GLES30.glClientWaitSync(
+                prev.fence,
+                GLES30.GL_SYNC_FLUSH_COMMANDS_BIT,
+                FENCE_TIMEOUT_NS,
+            )
+        if (waitResult == GLES30.GL_TIMEOUT_EXPIRED || waitResult == GLES30.GL_WAIT_FAILED) {
+            Log.w(TAG, "fence wait returned 0x${waitResult.toString(16)}; handing off anyway")
+        }
+        GLES30.glDeleteSync(prev.fence)
 
-    if (debugTiming && prev.submitNanos != 0L) {
-      // CPU-observed latency from the predecessor's GPU submit to the point
-      // its fence resolved (now). EXT_disjoint_timer_query (GL_TIME_ELAPSED_EXT
-      // + glGetQueryObjectui64v) has no android.opengl Java binding, so this
-      // wall-clock delta is the fallback; it measures submit->complete latency
-      // rather than pure on-GPU time, but reads a frame late exactly as a timer
-      // query would and adds no extra blocking wait.
-      val elapsedMs = (System.nanoTime() - prev.submitNanos) / 1_000_000.0
-      Log.d(PERF_TAG, "$timingLabel gpu submit->complete ~%.2f ms (nanoTime fallback)".format(elapsedMs))
-    }
+        if (debugTiming && prev.submitNanos != 0L) {
+            // CPU-observed latency from the predecessor's GPU submit to the point
+            // its fence resolved (now). EXT_disjoint_timer_query (GL_TIME_ELAPSED_EXT
+            // + glGetQueryObjectui64v) has no android.opengl Java binding, so this
+            // wall-clock delta is the fallback; it measures submit->complete latency
+            // rather than pure on-GPU time, but reads a frame late exactly as a timer
+            // query would and adds no extra blocking wait.
+            val elapsedMs = (System.nanoTime() - prev.submitNanos) / 1_000_000.0
+            Log.d(PERF_TAG, "$timingLabel gpu submit->complete ~%.2f ms (nanoTime fallback)".format(elapsedMs))
+        }
 
-    return Ready(prev.textureId, prev.width, prev.height, prev.rotation, prev.timestampNs)
-  }
+        return Ready(prev.textureId, prev.width, prev.height, prev.rotation, prev.timestampNs)
+    }
 
-  /**
-   * Free the last un-handed-off frame's texture and fence. Call on the GL
-   * thread from a processor teardown path. (No VideoFrameProcessor teardown
-   * hook exists today, mirroring Mask.release; provided for correctness.)
-   */
-  fun dispose() {
-    val p = pending ?: return
-    pending = null
-    try {
-      GLES30.glDeleteSync(p.fence)
-      GLES30.glDeleteTextures(1, intArrayOf(p.textureId), 0)
-    } catch (t: Throwable) {
-      Log.w(TAG, "dispose encountered an error; resources may leak", t)
+    /**
+     * Free the last un-handed-off frame's texture and fence. Call on the GL
+     * thread from a processor teardown path. (No VideoFrameProcessor teardown
+     * hook exists today, mirroring Mask.release; provided for correctness.)
+     */
+    fun dispose() {
+        val p = pending ?: return
+        pending = null
+        try {
+            GLES30.glDeleteSync(p.fence)
+            GLES30.glDeleteTextures(1, intArrayOf(p.textureId), 0)
+        } catch (t: Throwable) {
+            Log.w(TAG, "dispose encountered an error; resources may leak", t)
+        }
     }
-  }
 
-  /** The previous frame's texture + metadata, ready to wrap downstream. */
-  class Ready(
-    val textureId: Int,
-    val width: Int,
-    val height: Int,
-    val rotation: Int,
-    val timestampNs: Long,
-  )
+    /** The previous frame's texture + metadata, ready to wrap downstream. */
+    class Ready(
+        val textureId: Int,
+        val width: Int,
+        val height: Int,
+        val rotation: Int,
+        val timestampNs: Long,
+    )
 
-  companion object {
-    private const val TAG = "Kaleidoscope.Pipeline"
-    private const val PERF_TAG = "Perf"
+    companion object {
+        private const val TAG = "Kaleidoscope.Pipeline"
+        private const val PERF_TAG = "Perf"
 
-    // 2 ms. A signaled fence returns immediately; this only bounds the
-    // pathological case where the predecessor's GPU work has not finished a
-    // full frame later. glClientWaitSync takes nanoseconds.
-    private const val FENCE_TIMEOUT_NS = 2_000_000L
-  }
+        // 2 ms. A signaled fence returns immediately; this only bounds the
+        // pathological case where the predecessor's GPU work has not finished a
+        // full frame later. glClientWaitSync takes nanoseconds.
+        private const val FENCE_TIMEOUT_NS = 2_000_000L
+    }
 }

package/android/src/main/java/com/simiancraft/kaleidoscope/gpu/GlDebug.kt

@@ -12,30 +12,30 @@ import android.opengl.GLES30
 import android.util.Log
 
 internal object GlDebug {
-  private const val TAG = "Kaleidoscope.GL"
+    private const val TAG = "Kaleidoscope.GL"
 
-  /**
-   * Drain any pending GL errors and log them tagged with `stage`. Returns
-   * true if at least one error was found.
-   */
-  fun check(stage: String): Boolean {
-    var found = false
-    while (true) {
-      val err = GLES30.glGetError()
-      if (err == GLES30.GL_NO_ERROR) break
-      Log.e(TAG, "[$stage] glGetError = 0x${err.toString(16)} (${errorName(err)})")
-      found = true
+    /**
+     * Drain any pending GL errors and log them tagged with `stage`. Returns
+     * true if at least one error was found.
+     */
+    fun check(stage: String): Boolean {
+        var found = false
+        while (true) {
+            val err = GLES30.glGetError()
+            if (err == GLES30.GL_NO_ERROR) break
+            Log.e(TAG, "[$stage] glGetError = 0x${err.toString(16)} (${errorName(err)})")
+            found = true
+        }
+        return found
     }
-    return found
-  }
 
-  private fun errorName(err: Int): String =
-    when (err) {
-      GLES30.GL_INVALID_ENUM -> "GL_INVALID_ENUM"
-      GLES30.GL_INVALID_VALUE -> "GL_INVALID_VALUE"
-      GLES30.GL_INVALID_OPERATION -> "GL_INVALID_OPERATION"
-      GLES30.GL_INVALID_FRAMEBUFFER_OPERATION -> "GL_INVALID_FRAMEBUFFER_OPERATION"
-      GLES30.GL_OUT_OF_MEMORY -> "GL_OUT_OF_MEMORY"
-      else -> "unknown"
-    }
+    private fun errorName(err: Int): String =
+        when (err) {
+            GLES30.GL_INVALID_ENUM -> "GL_INVALID_ENUM"
+            GLES30.GL_INVALID_VALUE -> "GL_INVALID_VALUE"
+            GLES30.GL_INVALID_OPERATION -> "GL_INVALID_OPERATION"
+            GLES30.GL_INVALID_FRAMEBUFFER_OPERATION -> "GL_INVALID_FRAMEBUFFER_OPERATION"
+            GLES30.GL_OUT_OF_MEMORY -> "GL_OUT_OF_MEMORY"
+            else -> "unknown"
+        }
 }