GLSL functions are always named "main", and have no need for the DX version param. This currently means having two unneeded parameters in the addHighLevelShader call per shader type (vert, frag, geom - up to 6 unnecessary params).
It would be more elegant not to require those params - to have two (one for vert+frag, one for v+f+g) shorthand function wrappers in Irrlicht.
This is somewhat easier to work around client-side than the float* int* cast, but still this would be an improvement in the API
[fixed] Request for a shorthand addHighLevelShader for GL
[fixed] Request for a shorthand addHighLevelShader for GL
Last edited by hendu on Wed Jan 04, 2012 2:19 pm, edited 1 time in total.
Re: Request for a shorthand addHighLevelShader for GL
Ping. I'm willing to post patches, but please give me a yea/nay
Re: Request for a shorthand addHighLevelShader for GL
*sigh* we really need someone again in the team who cares mainly about Shaders... it does not seem like BlindSide will come back. I can say unfortunately nothing at all about it, never coded a Shader myself, so have to leave this for now to Hybrid.
IRC: #irrlicht on irc.libera.chat
Code snippet repository: https://github.com/mzeilfelder/irr-playground-micha
Free racer made with Irrlicht: http://www.irrgheist.com/hcraftsource.htm
Code snippet repository: https://github.com/mzeilfelder/irr-playground-micha
Free racer made with Irrlicht: http://www.irrgheist.com/hcraftsource.htm
-
- Admin
- Posts: 14143
- Joined: Wed Apr 19, 2006 9:20 pm
- Location: Oldenburg(Oldb), Germany
- Contact:
Re: Request for a shorthand addHighLevelShader for GL
Yes, that's ok. I even have some patches for similar things. Please send in a patch for this one.
Re: Request for a shorthand addHighLevelShader for GL
Ping as the holidays are over.