Use GL values for ARRAY_BUFFER and ELEMENT_ARRAY_BUFFER

[pjcozzi]

This may already been a known issue, but these should be the GL enum values, not strings:

        "bufferView_29": {
            "buffer": "duck",
            "byteLength": 76768,
            "byteOffset": 0,
            "target": "ARRAY_BUFFER"
        },
        "bufferView_30": {
            "buffer": "duck",
            "byteLength": 25272,
            "byteOffset": 76768,
            "target": "ELEMENT_ARRAY_BUFFER"
        },

From the WebGL spec:

const GLenum ARRAY_BUFFER                   = 0x8892;
const GLenum ELEMENT_ARRAY_BUFFER           = 0x8893;

[pjcozzi]

Likewise, we should use GL enum values for the primitive type.

            "primitives": [
                {
                    "indices": "indices_21",
                    "material": "blinn3-fx",
                    "primitive": "TRIANGLES",

From the WebGL spec:

    const GLenum POINTS                         = 0x0000;
    const GLenum LINES                          = 0x0001;
    const GLenum LINE_LOOP                      = 0x0002;
    const GLenum LINE_STRIP                     = 0x0003;
    const GLenum TRIANGLES                      = 0x0004;
    const GLenum TRIANGLE_STRIP                 = 0x0005;
    const GLenum TRIANGLE_FAN                   = 0x0006;

[fabrobinet]

That's done on master no ? Or at least on dev-2

[pjcozzi]

It's not in master, or at least not in the example models. Sorry, I'm not brave enough to develop against dev-2.

[pjcozzi]

This is fixed in master, but the example models were not updated. This is sloppy of us. We need to keep these in sync so we know what changes version to version. Diffing the examples is the very first thing I do when updating the converter.

[pjcozzi]

Also, target is fixed, but not primitive.

[fabrobinet]

Ok, let's keep it for that then

[fabrobinet]

Yes, about updating models... obviously my bad... I'll update then as soon as I can.

[fabrobinet]

same here, fixed on dev-2.

[fabrobinet]

now on master.