Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

RFC: add classic tests with as much ::Modern usage as possible #41

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

wchristian
Copy link
Collaborator

Don't merge this! I want input.

Since the test suite at the moment is rather small and we have an older piece of perfectly good code lying around, i figured i might as well port it to ::Modern as much as possible, and make it a highly optional test that we can use to verify base functionality.

Additionally this serves as a neat additional proving ground for which functions you wish to retain from the older set and which of the functions in the test you'd change to other calls. Please look through it and primarily comment on that. (Though i welcome any other thoughts as well.)

@wchristian wchristian force-pushed the oldschool_tests branch 2 times, most recently from 1a5f4b3 to 191a1ae Compare February 16, 2017 15:47
@wchristian
Copy link
Collaborator Author

Migrated the glut functions to OpenGL::GLUT as per Perl-GPU/OpenGL-GLUT#2 and everything seems to be fine. This leaves:

use OpenGL qw' glpCheckExtension glpFullScreen glpRestoreScreen ',    # glp

  qw' gluBuild2DMipmaps_c gluPerspective gluOrtho2D gluUnProject_p gluErrorString ',    # glu

  qw' glProgramStringARB_p
  glMapBufferARB_p glGetBufferSubDataARB_p
  glLightfv_p
  glDeleteProgramsARB_p glDeleteFramebuffersEXT_p glDeleteRenderbuffersEXT_p glDeleteBuffersARB_p glDeleteTextures_p
  glGetBufferParameterivARB_p glGetProgramStringARB_p glGetProgramivARB_p glGetProgramEnvParameterdvARB_p glGetProgramEnvParameterfvARB_p
  glVertexPointer_p glColorPointer_p glTexCoordPointer_p glNormalPointer_p

  glTexImage2D_s glTexImage2D_p
  ';

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant