Skip to content

Fix for "Auto" noise reduction setting - #339

Merged
drewnoakes merged 1 commit into
drewnoakes:masterfrom
davidekholm:patch-1
Apr 15, 2018
Merged

Fix for "Auto" noise reduction setting#339
drewnoakes merged 1 commit into
drewnoakes:masterfrom
davidekholm:patch-1

Conversation

@davidekholm

Copy link
Copy Markdown
Contributor

Olympus cameras with the "Auto" noise reduction setting (bit 3 set) causes this code to crash with an IndexOutOfBoundsException.
I've now added support for bit 3 ("Auto") and made the code fault tolerant if future bits are added.

Reference: https://sno.phy.queensu.ca/~phil/exiftool/TagNames/Olympus.html

I add a sample image so you can reproduce the bug and verify the fix I made too.
olympus auto noise reduction

Olympus cameras with the "Auto" noise reduction setting  (bit 3 set) causes this code to crash with an IndexOutOfBoundsException.
I've now added support for bit 3 ("Auto") and made the code fault tolerant if future bits are added.

Reference: https://sno.phy.queensu.ca/~phil/exiftool/TagNames/Olympus.html
@drewnoakes
drewnoakes merged commit 8de2330 into drewnoakes:master Apr 15, 2018
@drewnoakes

Copy link
Copy Markdown
Owner

Thanks David. I'll add the image to the sample library too.

drewnoakes added a commit to drewnoakes/metadata-extractor-images that referenced this pull request Apr 15, 2018
Nadahar pushed a commit to Nadahar/metadata-extractor-images that referenced this pull request Oct 4, 2018
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