Skip to content

fix(raw): include LibRaw error messages within ours - #5573

Merged
lgritz merged 2 commits into
AcademySoftwareFoundation:mainfrom
lgritz:lg-thumb
Oct 17, 2026
Merged

lgritz merged 2 commits into
AcademySoftwareFoundation:mainfrom
lgritz:lg-thumb

Conversation

@lgritz

@lgritz lgritz commented Oct 16, 2026

Copy link
Copy Markdown
Collaborator

There were some kinds of LibRaw failures (particularly involving thumbnail retrieval) where we correctly returned false and gave an error message, but there was an error code from libraw that could give more information and we weren't using it. Add their error message within ours.

There were some kinds of LibRaw failures (particularly involving
thumbnail retrieval) where we correctly returned false and gave an
error message, but there was an error code from libraw that could give
more information and we weren't using it. Add their error message
within ours.

Signed-off-by: Larry Gritz <lg@larrygritz.com>
@lgritz
lgritz requested a review from jinhgkim October 16, 2026 01:25
Comment thread src/raw.imageio/rawinput.cpp Outdated
Comment thread src/raw.imageio/rawinput.cpp Outdated
Comment thread src/raw.imageio/rawinput.cpp
Comment thread src/raw.imageio/rawinput.cpp Outdated
…e and libraw-specific codes.

Signed-off-by: Larry Gritz <lg@larrygritz.com>

@jinhgkim jinhgkim left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks!

@lgritz
lgritz merged commit e14e9a4 into AcademySoftwareFoundation:main Oct 17, 2026
30 checks passed
@lgritz
lgritz deleted the lg-thumb branch October 17, 2026 00:52
lgritz added a commit to lgritz/OpenImageIO that referenced this pull request Oct 17, 2026
…oundation#5573)

There were some kinds of LibRaw failures (particularly involving
thumbnail retrieval) where we correctly returned false and gave an error
message, but there was an error code from libraw that could give more
information and we weren't using it. Add their error message within
ours.

---------

Signed-off-by: Larry Gritz <lg@larrygritz.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants