[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