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

more of the perlish syntax #60

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

more of the perlish syntax #60

wants to merge 2 commits into from

Conversation

dk
Copy link
Contributor

@dk dk commented Jun 26, 2017

Hi Chris,

Okay, as we discussed the perlishness of the API, here's my first shot at what I would consider 'perlish enough' for my needs. I'm not really sure how it fits into your vision you have told me about yesteday, but at least that's how I understood it. You're welcome to hack it further to whatever form you deem necessary.

Dmitry

@devel-chm
Copy link
Collaborator

Yes, that is the general idea. However, there are some implementation details that are mostly related to issues of API stability and compatibility. The main one being that I don't want to have the basename implementation (e.g. glGetActiveUniform) used until the API is finalized. In the interim, the various steps would be as suffixed names (e.g. glGetActiveUniform_p or glGetActiveUniform_dmitry...) so that the changes in OpenGL::Modern::Helpers won't be overwriting the stable bindings.

From the point of view of availability of perlish bindings, the only difference is the routine would have a suffixed name. If it becomes the accepted, preferred implementation then we would add the more standard name. The implementation of generic type and counts for some of the glXxx[234][dusl]v routines as a glXxxv routine is definitely something of interest for a friendly perlish binding. At the moment, I'm keeping closer to the C API until things are stable so that the routines are consistent even if not very perlish.

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.

2 participants