jogl
  1. jogl
  2. JOGL-374

GL.GL_CLIENT_ALL_ATTRIB_BITS should be type int

    Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: current
    • Fix Version/s: milestone 1
    • Component/s: jogl
    • Labels:
      None
    • Environment:

      Operating System: All
      Platform: All

    • Issuezilla Id:
      374

      Description

      GL.GL_CLIENT_ALL_ATTRIB_BITS is currently type long. It should be type int.

        Activity

        Hide
        kbr added a comment -

        The use of long for this value is deliberate because of the lack of unsigned integer values in the Java
        language. GlueGen automatically picks the next larger type when the sign bit is set, as in this value. I
        don't remember exactly when we made the change to GlueGen but I am pretty sure it was motivated by
        incorrect glue code generation for some set of C APIs, possibly OpenGL APIs. There is no plan to change
        this unless there is a strong reason to do so.

        Show
        kbr added a comment - The use of long for this value is deliberate because of the lack of unsigned integer values in the Java language. GlueGen automatically picks the next larger type when the sign bit is set, as in this value. I don't remember exactly when we made the change to GlueGen but I am pretty sure it was motivated by incorrect glue code generation for some set of C APIs, possibly OpenGL APIs. There is no plan to change this unless there is a strong reason to do so.
        Hide
        kitfox added a comment -

        Well, right now you can't pass it to GL.glPushClientAttrib(), which only accepts
        integers. I presume that casting it to int would chop off some important bits.

        Show
        kitfox added a comment - Well, right now you can't pass it to GL.glPushClientAttrib(), which only accepts integers. I presume that casting it to int would chop off some important bits.

          People

          • Assignee:
            jogl-issues
            Reporter:
            kitfox
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated: