[Bf-committers] Patch: OCIO Parse Colorspace From Filename
Dalai Felinto
dfelinto at gmail.com
Mon Dec 11 11:32:52 CET 2017
Hi Troy,
> https://developer.blender.org/differential/diff/9682/
Could you submit the patch with `arc diff` instead? I have a few points to
make to the patch, and I would like to try it.
For instance:
1. Did you make sure Blender treat this texture as a "non-color" texture
for my_texture_linear.tiff?
2. You seem to be handling linear and srgb only. However you mentioned
acescg here as well which doesn't seem to be handled.
3. Code-style:
Basically follow: https://wiki.blender.org/index.php/Dev:Doc/Code_Style
* use `if (` instead of `if(`
* use explicit {}
* configParseColorSpaceFromString seems to have way to many left spaces
before the filename.
Thanks,
Dalai
More information about the Bf-committers
mailing list