Adopt zero-length interface members from Objectively

[jdolan]

Mechanical follow-through of jdolan/Objectively#41, which removes the per-instance interface pointer: $ now resolves the interface through Object::clazz, and the struct member exists only so typeof can name the interface type. ClassDef.interfaceOffset no longer exists.

Two edits per class, no call-site changes:

• XInterface *interface; → XInterface *interface[0]; (15 classes + Copilot guidance)
• .interfaceOffset = offsetof(X, interface), removed from each ClassDef

Verified: builds clean (0 warnings) against Objectively zero-length-interface installed to /usr/local; Examples/Hello and Examples/HelloCompute build. ObjectivelyGPU has no make check suite.

Depends on jdolan/Objectively#41 being installed first. ObjectivelyMVC follows on top of this.

🤖 Generated with Claude Code

[Copilot]

🟢 Approval recommended

The changes are consistent across all affected classes, remove all interfaceOffset usage, and no remaining per-instance ->interface accesses were found.

Pull request overview

Aligns ObjectivelyGPU’s class layouts with the updated Objectively runtime (jdolan/Objectively#41) by removing per-instance interface pointers and relying on Object::clazz for vtable resolution, while keeping a zero-length interface member solely for typeof/type naming.

Changes:

• Replace per-instance XInterface *interface; members with XInterface *interface[0]; across all ObjectivelyGPU classes.
• Remove .interfaceOffset = offsetof(X, interface), from each ClassDef initializer (field no longer exists upstream).
• Update .github/copilot-instructions.md to document the new class-layout pattern.

File summaries

File	Description
Sources/ObjectivelyGPU/TransferBuffer.h	Switch interface member to zero-length array; clarify Doxygen brief.
Sources/ObjectivelyGPU/TransferBuffer.c	Remove ClassDef.interfaceOffset initializer entry.
Sources/ObjectivelyGPU/Texture.h	Switch interface member to zero-length array; clarify Doxygen brief.
Sources/ObjectivelyGPU/Texture.c	Remove ClassDef.interfaceOffset initializer entry.
Sources/ObjectivelyGPU/Shader.h	Switch interface member to zero-length array; clarify Doxygen brief.
Sources/ObjectivelyGPU/Shader.c	Remove ClassDef.interfaceOffset initializer entry.
Sources/ObjectivelyGPU/Sampler.h	Switch interface member to zero-length array; clarify Doxygen brief.
Sources/ObjectivelyGPU/Sampler.c	Remove ClassDef.interfaceOffset initializer entry.
Sources/ObjectivelyGPU/RenderPass.h	Switch interface member to zero-length array; clarify Doxygen brief.
Sources/ObjectivelyGPU/RenderPass.c	Remove ClassDef.interfaceOffset initializer entry.
Sources/ObjectivelyGPU/RenderDevice.h	Switch interface member to zero-length array; clarify Doxygen brief.
Sources/ObjectivelyGPU/RenderDevice.c	Remove ClassDef.interfaceOffset initializer entry.
Sources/ObjectivelyGPU/QueryPool.h	Switch interface member to zero-length array; clarify Doxygen brief.
Sources/ObjectivelyGPU/QueryPool.c	Remove ClassDef.interfaceOffset initializer entry.
Sources/ObjectivelyGPU/GraphicsPipeline.h	Switch interface member to zero-length array; clarify Doxygen brief.
Sources/ObjectivelyGPU/GraphicsPipeline.c	Remove ClassDef.interfaceOffset initializer entry.
Sources/ObjectivelyGPU/Framebuffer.h	Switch interface member to zero-length array; clarify Doxygen brief.
Sources/ObjectivelyGPU/Framebuffer.c	Remove ClassDef.interfaceOffset initializer entry.
Sources/ObjectivelyGPU/Fence.h	Switch interface member to zero-length array; clarify Doxygen brief.
Sources/ObjectivelyGPU/Fence.c	Remove ClassDef.interfaceOffset initializer entry.
Sources/ObjectivelyGPU/CopyPass.h	Switch interface member to zero-length array; clarify Doxygen brief.
Sources/ObjectivelyGPU/CopyPass.c	Remove ClassDef.interfaceOffset initializer entry.
Sources/ObjectivelyGPU/ComputePipeline.h	Switch interface member to zero-length array; clarify Doxygen brief.
Sources/ObjectivelyGPU/ComputePipeline.c	Remove ClassDef.interfaceOffset initializer entry.
Sources/ObjectivelyGPU/ComputePass.h	Switch interface member to zero-length array; clarify Doxygen brief.
Sources/ObjectivelyGPU/ComputePass.c	Remove ClassDef.interfaceOffset initializer entry.
Sources/ObjectivelyGPU/CommandBuffer.h	Switch interface member to zero-length array; clarify Doxygen brief.
Sources/ObjectivelyGPU/CommandBuffer.c	Remove ClassDef.interfaceOffset initializer entry.
Sources/ObjectivelyGPU/Buffer.h	Switch interface member to zero-length array; clarify Doxygen brief.
Sources/ObjectivelyGPU/Buffer.c	Remove ClassDef.interfaceOffset initializer entry.
.github/copilot-instructions.md	Update documented class pattern to use interface[0] and omit interfaceOffset.

Review details

• Files reviewed: 31/31 changed files
• Comments generated: 0
• Review effort level: Lite

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.