Changed snprintf formatting to satisfy some compilers.

[pellerington]

Signed-off-by: Peter Ellerington elleringtonp@gmail.com

Description

When building OSL my compiler (GCC 11.3 on Ubuntu) was complaining about these snprintfs format strings not being a string literal.
Writing them to a "%s" seems to satisfy it and should produce the same result.

Tests

no changes in tests

Checklist:

• I have read the contribution guidelines.
• I have previously submitted a Contributor License Agreement.
• I have updated the documentation, if applicable.
• I have ensured that the change is tested somewhere in the testsuite (adding new test cases if necessary).
• My code follows the prevailing code style of this project.

[linux-foundation-easycla]

The committers listed above are authorized under a signed CLA.

• ✅ login: pellerington (5a3bb71)

[lgritz]

Oh wow, yes. Thanks for the fix.

Can you please get the CLA squared away, and then I'll merge this.

[pellerington]

Okay, I think I've done everything now.

I've made some other modifications to make integrating my renderer easier. Let me know if your interested!

[lgritz]

Sure, what other modifications did you have in mind?

[pellerington]

The main change I added was a optional register closure which lets that closure allocate with a unique function.
Eg. for microfacet instead of "osl_allocate_closure_component(...)" being run a function of the form "osl_microfacet(sg, weight, distribution, normal, roughnes ... )" gets run.

It's slightly more work because the renderer needs to implement a function for each closure and copy the arguments more manually, but there's much more freedom.

It let me do processing/interfacing with my renderer memory directly in that function (rather than storing args then looping back over everything after OSL had run).
It also meant I didn't have to deal with ClosureComponent data and offset calculations/alignment issues on the gpu. I could just define a class (eg. ClosureMicrofacet) and copy the data there.
Also a simpler register closure function because you just have to list the args (not their alignment/size in the struct).