Fix excessive glObjectLabel calls for vertex/index buffers

[Copilot]

glObjectLabel was being called in setVertexAttrib on every render update for every vertex buffer, causing measurable CPU overhead visible in profiles and RenderDoc. Index buffers were never labeled at all since setVertexAttrib rejects them early.

Changes

• updateBufferData(VertexBuffer): Add glObjectLabel call immediately after buffer creation (bufId == -1 path), consistent with how shaders are labeled at creation time
• setVertexAttrib: Remove the per-frame glObjectLabel call entirely
• Drop the vb.getName() != null guard — VertexBuffer.getName() never returns null (falls back to a generated name like VertexBuffer(Position))

Index buffers now get labeled correctly as a natural consequence of the move — they pass through updateBufferData just like vertex buffers.

// Before: called every frame in setVertexAttrib
if (debug && caps.contains(Caps.GLDebug)) {
    if (vb.getName() != null) glext.glObjectLabel(GLExt.GL_BUFFER, vb.getId(), vb.getName());
}

// After: called once at creation in updateBufferData (covers both VBOs and IBOs)
if (debug && caps.contains(Caps.GLDebug)) {
    glext.glObjectLabel(GLExt.GL_BUFFER, bufId, vb.getName());
}

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• dl.google.com
  • Triggering command: /usr/lib/jvm/temurin-17-jdk-amd64/bin/java /usr/lib/jvm/temurin-17-jdk-amd64/bin/java --add-opens=java.base/java.lang=ALL-UNNAMED --add-opens=java.base/java.lang.invoke=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.prefs/java.util.prefs=ALL-UNNAMED --add-exports=jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED --add-exports=jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.prefs/java.util.prefs=ALL-UNNAMED --add-opens=java.base/java.nio.charset=ALL-UNNAMED --add-opens=java.base/java.net=ALL-UNNAMED --add-opens=java.base/java.util.concurrent.atomic=ALL-UNNAMED --add-opens=java.xml/javax.xml.namespace=ALL-UNNAMED -XX:MaxMetaspaceSize=384m -XX:+HeapDumpOnOutOfMemoryError -Xms256m -Xmx512m -Dfile.encoding=UTF-8 -Duser.country -Duser.language=en (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

---

💬 Send tasks to Copilot coding agent from Slack and Teams to turn conversations into code. Copilot posts an update in your thread when it's finished.

[Trass3r]

It's not that easy due to GL's bind-to-create concept.
See Trass3r@f994546.