ziglibs / zgl

Zig OpenGL Wrapper
MIT License
425 stars 62 forks source link

BufferMapTarget duplicates BufferTarget #83

Closed mondegreengames closed 1 year ago

mondegreengames commented 1 year ago

zgl.zig contains definitions for both BufferMapTarget and BufferTarget with the same contents. And buffer functions aren't consistent with which one to use. Example: mapBuffer() uses BufferMapTarget and mapBufferRange() uses BufferTarget.

IMO, BufferTarget seems like the one to keep. It's used more than BufferMapTarget and existed first and is commented. The usages of BufferMapTarget in mapBuffer() and unmapBuffer() could be changed to use BufferTarget instead.