fix(exr): allow an empty "name" metadata to be read (#4536)

lgritz · web-flow · commit c5c0339b2720 · 2024-11-15T13:16:02.000-08:00
Continuation of PR #4528. I forgot to change the spot in the OpenEXR "core" API as well. Signed-off-by: Larry Gritz <lg@larrygritz.com>
diff --git a/src/openexr.imageio/exrinput_c.cpp b/src/openexr.imageio/exrinput_c.cpp
@@ -757,7 +757,7 @@ OpenEXRCoreInput::PartInfo::parse_header(OpenEXRCoreInput* in,
     // EXR "name" also gets passed along as "oiio:subimagename".
     const char* partname;
     if (exr_get_name(ctxt, subimage, &partname) == EXR_ERR_SUCCESS) {
-        if (partname && partname[0] != '\0')
+        if (partname)
             spec.attribute("oiio:subimagename", partname);
     }
 

Original file line number	Diff line number	Diff line change
`@@ -757,7 +757,7 @@ OpenEXRCoreInput::PartInfo::parse_header(OpenEXRCoreInput* in,`
`757`	`757`	`// EXR "name" also gets passed along as "oiio:subimagename".`
`758`	`758`	`const char* partname;`
`759`	`759`	`if (exr_get_name(ctxt, subimage, &partname) == EXR_ERR_SUCCESS) {`
`760`		`- if (partname && partname[0] != '\0')`
	`760`	`+ if (partname)`
`761`	`761`	`spec.attribute("oiio:subimagename", partname);`
`762`	`762`	`}`
`763`	`763`