Hi Marc,

> please see the attached patch for the new module.

Very nice. Looks all good, except for some nit-picking in the comments:

> +     Introspection:      ELEMENT *base = stack_base (&stack);

I would add a comment here:
  Where ELEMENT is the type to which GL_STACK_ELEMENT was defined when
  this file was included.
(It would be tempting to write
    GL_STACK_ELEMENT *base = stack_base (&stack);
but GL_STACK_ELEMENT is no longer defined after the file was included...)

Other than that, please feel free to commit and push this. Thanks!!

Bruno


Reply via email to