Image nodes : Remove unnecessary use of OCIO configs

OCIO will look up ROLE_SCENE_LINEAR for us dynamically, so there's no need to do the lookup manually ourselves.
boberfly · Jun 2, 2023 · 12555f3 · 12555f3
1 parent 53e14c8
commit 12555f3
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 6 deletions.
diff --git a/src/GafferImage/ImageReader.cpp b/src/GafferImage/ImageReader.cpp
@@ -165,8 +165,7 @@ ImageReader::ImageReader( const std::string &name )
  addChild( colorSpace );
  colorSpace->inPlug()->setInput( oiioReader->outPlug() );
  colorSpace->inputSpacePlug()->setInput( intermediateColorSpacePlug() );
- OCIO_NAMESPACE::ConstConfigRcPtr config = OCIO_NAMESPACE::GetCurrentConfig();
- colorSpace->outputSpacePlug()->setValue( config->getColorSpace( OCIO_NAMESPACE::ROLE_SCENE_LINEAR )->getName() );
+ colorSpace->outputSpacePlug()->setValue( OCIO_NAMESPACE::ROLE_SCENE_LINEAR );
  colorSpace->processUnpremultipliedPlug()->setValue( true );
  intermediateImagePlug()->setInput( colorSpace->outPlug() );
 }

diff --git a/src/GafferImage/ImageWriter.cpp b/src/GafferImage/ImageWriter.cpp
@@ -1525,10 +1525,7 @@ ImageWriter::ImageWriter( const std::string &name )
  ColorSpacePtr colorSpaceChild = new ColorSpace( "__colorSpace" );
  addChild( colorSpaceChild );
 
- OCIO_NAMESPACE::ConstConfigRcPtr config = OCIO_NAMESPACE::GetCurrentConfig();
-
- colorSpaceChild->inputSpacePlug()->setValue( config->getColorSpace( OCIO_NAMESPACE::ROLE_SCENE_LINEAR )->getName() );
-
+ colorSpaceChild->inputSpacePlug()->setValue( OCIO_NAMESPACE::ROLE_SCENE_LINEAR );
  colorSpaceChild->outputSpacePlug()->setValue( "${__imageWriter:colorSpace}" );
 
  ValuePlugPtr layoutPlug = new ValuePlug( "layout" );