Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[xml] ColorTableParameterPNameSGI group used but not defined #356

Closed
SunSerega opened this issue Jan 26, 2020 · 1 comment · Fixed by #375
Closed

[xml] ColorTableParameterPNameSGI group used but not defined #356

SunSerega opened this issue Jan 26, 2020 · 1 comment · Fixed by #375
Assignees

Comments

@SunSerega
Copy link
Contributor

SunSerega commented Jan 26, 2020

        <command>
            <proto>void <name>glColorTableParameterfv</name></proto>
            <param group="ColorTableTarget"><ptype>GLenum</ptype> <name>target</name></param>
            <param group="ColorTableParameterPNameSGI"><ptype>GLenum</ptype> <name>pname</name></param>
            <param group="CheckedFloat32" len="COMPSIZE(pname)">const <ptype>GLfloat</ptype> *<name>params</name></param>
            <glx type="render" opcode="2054"/>
        </command>

(this is only one of functions using ColorTableParameterPNameSGI)

I think these are the enums that should be passed as ColorTableParameterPNameSGI:

    COLOR_TABLE_SCALE_SGI                0x80D6
     COLOR_TABLE_BIAS_SGI                0x80D7

But this info should be further checked before applying it.

@Perksey
Copy link
Contributor

Perksey commented Jan 26, 2020

Thanks for reporting this! We always welcome PRs to help improve and maintain the correctness of the new-syntax groupings. If not done by a community member, this will be included next time we have a group sprint here at Ultz.

@oddhack oddhack added this to the Needs Action/PR milestone Jan 27, 2020
Perksey added a commit to Perksey/OpenGL-Registry that referenced this issue Mar 6, 2020
@Perksey Perksey mentioned this issue Mar 6, 2020
oddhack pushed a commit that referenced this issue Mar 11, 2020
* Fix #355

* Fix #356

* Fix #357

* Fix #358

* Add groups to grouped blocks, fixes #359
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants