[rlgl] rlLoadShaderBuffer() data clear just after data copying

[Ka-lua]

The buffer clearing is probably some debugging artifact? This line should be removed:

raylib/src/rlgl.h

Line 4102 in 2e00d16

glClearBufferData(GL_SHADER_STORAGE_BUFFER, GL_R8UI, GL_RED_INTEGER, GL_UNSIGNED_BYTE, 0);

[raysan5]

@Ka-lua Afaik, that line makes sure the loaded buffer is filled with 0, in case a NULL data buffer is created (as used in the provided example).

That code was contributed so I don't know if that's the best approach, is there any reason we don't want that kind of initialization for the buffer? Could there be any issue if we avoid that kind of initialization?

[Ka-lua]

That's right, it clears the buffer to 0 which might be what you want, but that makes the data parameter useless. You could do this to only clear the buffer if data is the null pointer:

glBufferData(GL_SHADER_STORAGE_BUFFER, size, data, usageHint? usageHint : RL_STREAM_COPY);
if (data == NULL)
    glClearBufferData(GL_SHADER_STORAGE_BUFFER, GL_R8UI, GL_RED_INTEGER, GL_UNSIGNED_BYTE, 0);