[fixed] Request for a shorthand addHighLevelShader for GL

You discovered a bug in the engine, and you are sure that it is not a problem of your code? Just post it in here. Please read the bug posting guidelines first.
Post Reply
hendu
Posts: 2600
Joined: Sat Dec 18, 2010 12:53 pm

[fixed] Request for a shorthand addHighLevelShader for GL

Post by hendu »

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 :)
Last edited by hendu on Wed Jan 04, 2012 2:19 pm, edited 1 time in total.
hendu
Posts: 2600
Joined: Sat Dec 18, 2010 12:53 pm

Re: Request for a shorthand addHighLevelShader for GL

Post by hendu »

Ping. I'm willing to post patches, but please give me a yea/nay :)
CuteAlien
Admin
Posts: 9734
Joined: Mon Mar 06, 2006 2:25 pm
Location: Tübingen, Germany
Contact:

Re: Request for a shorthand addHighLevelShader for GL

Post by CuteAlien »

*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
hybrid
Admin
Posts: 14143
Joined: Wed Apr 19, 2006 9:20 pm
Location: Oldenburg(Oldb), Germany
Contact:

Re: Request for a shorthand addHighLevelShader for GL

Post by hybrid »

Yes, that's ok. I even have some patches for similar things. Please send in a patch for this one.
hendu
Posts: 2600
Joined: Sat Dec 18, 2010 12:53 pm

Re: Request for a shorthand addHighLevelShader for GL

Post by hendu »

Ping as the holidays are over.
Post Reply