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

Mix command arguments #1

Open
fred1024 opened this issue Jun 29, 2015 · 2 comments
Open

Mix command arguments #1

fred1024 opened this issue Jun 29, 2015 · 2 comments
Labels

Comments

@fred1024
Copy link

Hi Saharan, amazing work, i spent ages looking for something like OGSL. Very happy to have found your excellent project.
I had a crash with the mix command "incorrect number of arguments in calling function mix: expected 2 but given 3"
So I changed in OGSL in analyzeMain()

addBuiltInFunction(OGSLConstants.BUILT_IN_MIX, 2);
to
addBuiltInFunction(OGSLConstants.BUILT_IN_MIX, 3);

works like a charm, i thought I should let you know.
You did a real good work, thanks for making your work available

@saharan
Copy link
Owner

saharan commented Jul 19, 2015

Hi fred1024,
Thank you for very useful information! I've just fixed it.

@fred1024
Copy link
Author

No probs ... it's just amazing your work.
I have tried to modify a bit ogsl so that it can takes a mapping of the constants positions/order I want for a constant. I am not too satisfied with what i'have done (a bit of a ugly hack) but it works and it would be cool if you upgrade ogsl to offer this possibility. Anyway I love what you have done it beats everything I have comme across for shaders in flash.

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

No branches or pull requests

2 participants