dgvisioscopie's picture

Problem between GL.ColorTable and Version12 enum

Project:The Open Toolkit library
Version:0.9.x-dev
Component:Code
Category:bug report
Priority:critical
Assigned:Unassigned
Status:closed
Description

In GL.ColorTable, the target must be GL_COLOR_TABLE(...) as shown downer.
In the function declaration, target parameter is defined as Version12 enum value.

Version12 doesn't contain the needed values (GL_COLOR_TABLE ...), but GL.EnableCap does.

        // Résumé :
        //     Define a color lookup table
        //
        // Paramètres :
        //   target:
        //      Must be one of GL_COLOR_TABLE, GL_POST_CONVOLUTION_COLOR_TABLE, GL_POST_COLOR_MATRIX_COLOR_TABLE,
        //     GL_PROXY_COLOR_TABLE, GL_PROXY_POST_CONVOLUTION_COLOR_TABLE, or GL_PROXY_POST_COLOR_MATRIX_COLOR_TABLE.
        //
        //   internalformat:
        //      The internal format of the color table. The allowable values are GL_ALPHA,
        //     GL_ALPHA4, GL_ALPHA8, GL_ALPHA12, GL_ALPHA16, GL_LUMINANCE, GL_LUMINANCE4,
        //     GL_LUMINANCE8, GL_LUMINANCE12, GL_LUMINANCE16, GL_LUMINANCE_ALPHA, GL_LUMINANCE4_ALPHA4,
        //     GL_LUMINANCE6_ALPHA2, GL_LUMINANCE8_ALPHA8, GL_LUMINANCE12_ALPHA4, GL_LUMINANCE12_ALPHA12,
        //     GL_LUMINANCE16_ALPHA16, GL_INTENSITY, GL_INTENSITY4, GL_INTENSITY8, GL_INTENSITY12,
        //     GL_INTENSITY16, GL_R3_G3_B2, GL_RGB, GL_RGB4, GL_RGB5, GL_RGB8, GL_RGB10,
        //     GL_RGB12, GL_RGB16, GL_RGBA, GL_RGBA2, GL_RGBA4, GL_RGB5_A1, GL_RGBA8, GL_RGB10_A2,
        //     GL_RGBA12, and GL_RGBA16.
        //
        //   width:
        //      The number of entries in the color lookup table specified by data.
        //
        //   format:
        //      The format of the pixel data in data. The allowable values are GL_RED, GL_GREEN,
        //     GL_BLUE, GL_ALPHA, GL_LUMINANCE, GL_LUMINANCE_ALPHA, GL_RGB, GL_BGR, GL_RGBA,
        //     and GL_BGRA.
        //
        //   type:
        //      The type of the pixel data in data. The allowable values are GL_UNSIGNED_BYTE,
        //     GL_BYTE, GL_UNSIGNED_SHORT, GL_SHORT, GL_UNSIGNED_INT, GL_INT, GL_FLOAT,
        //     GL_UNSIGNED_BYTE_3_3_2, GL_UNSIGNED_BYTE_2_3_3_REV, GL_UNSIGNED_SHORT_5_6_5,
        //     GL_UNSIGNED_SHORT_5_6_5_REV, GL_UNSIGNED_SHORT_4_4_4_4, GL_UNSIGNED_SHORT_4_4_4_4_REV,
        //     GL_UNSIGNED_SHORT_5_5_5_1, GL_UNSIGNED_SHORT_1_5_5_5_REV, GL_UNSIGNED_INT_8_8_8_8,
        //     GL_UNSIGNED_INT_8_8_8_8_REV, GL_UNSIGNED_INT_10_10_10_2, and GL_UNSIGNED_INT_2_10_10_10_REV.
        //
        //   data:
        //      Pointer to a one-dimensional array of pixel data that is processed to build
        //     the color table.
        [AutoGenerated(Category = "Version12", Version = "1.2", EntryPoint = "glColorTable")]
        public static void ColorTable(Version12 target, PixelInternalFormat internalformat, int width, PixelFormat format, PixelType type, IntPtr table);

It might be better to change Version12 type to int as

public static void ColorTable(int target, PixelInternalFormat internalformat, int width, PixelFormat format, PixelType type, IntPtr table);

Or to include all the missing values in Version12 enum.


Comments

Comment viewing options

Select your preferred way to display the comments and click "Save settings" to activate your changes.
the Fiddler's picture

#1

Version:0.9.7» 0.9.x-dev
Status:open» fixed

Thanks, the enums in question belong to SGI_color_table. Fixed in rev. 1736.

Please note that the work on OpenGL 1.2 tokens is not complete, mainly because the relevant functionality (ARB_imaging, EXT_convolution, etc) is not in demand. Any function that contains a parameter of "Version12", or any other VersionXY type for that matter, is considered a bug to be fixed as soon as it is reported.

the Fiddler's picture

#2

Status:fixed» closed

Closing issues resolved in 0.9.8.