Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename resp. remove duplicate exif keys #2434

Merged
merged 9 commits into from
Dec 30, 2022

Conversation

norbertwg
Copy link
Contributor

Duplicate key names with different tag are renamed. Following most of the existing entries, the key with higher tag number is renamed to ...2.

Exception: Canon.CustomFunctions, Olympus.ImageHeight, Olympus.ImageWidth. In the available images samples only the key with higher tag number is used, so changing the other has less impact on test data and also on users.

Canon.WhiteBalanceTable with tag 0x00a4 was listed twice, once with description "SharpnessTable". The entry with the deviating description is deleted.

Duplicate key names with different tag are renamed
Following most of the existing entries, the key with higher tag number
is renamed to ...2.
Exceptions: Canon.CustomFunctions,Olympus.ImageHeight,Olympus.ImageWidth
In the available images samples only the key with higher tag number is
used, so changing the other has less impact on test data and also on
users.
Canon.WhiteBalanceTable with tag 0x00a4 was listed twice, once with
description "SharpnessTable". The entry with the deviating description
is deleted.
@codecov
Copy link

codecov bot commented Dec 10, 2022

Codecov Report

Merging #2434 (e8c3855) into main (5bbd702) will increase coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2434      +/-   ##
==========================================
+ Coverage   65.43%   65.48%   +0.05%     
==========================================
  Files         118      118              
  Lines       21661    21661              
  Branches    10467    10467              
==========================================
+ Hits        14173    14184      +11     
+ Misses       5296     5280      -16     
- Partials     2192     2197       +5     
Impacted Files Coverage Δ
src/canonmn_int.cpp 73.48% <ø> (ø)
src/easyaccess.cpp 94.44% <ø> (ø)
src/minoltamn_int.cpp 69.46% <ø> (ø)
src/nikonmn_int.cpp 63.11% <ø> (ø)
src/olympusmn_int.cpp 44.68% <ø> (+2.12%) ⬆️
src/panasonicmn_int.cpp 53.50% <ø> (+4.38%) ⬆️
src/sonymn_int.cpp 80.48% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

kevinbackhouse
kevinbackhouse previously approved these changes Dec 27, 2022
@kmilos
Copy link
Collaborator

kmilos commented Dec 29, 2022

No idea why the CI fails, and if it's safe to merge... Maybe rebase to include the CI switch to Ubuntu 22.04?

@piponazo
Copy link
Collaborator

No idea why the CI fails, and if it's safe to merge... Maybe rebase to include the CI switch to Ubuntu 22.04?

Yes, the CI issue was fixed already. Please rebase your branch in top of main and push again.

@mergify mergify bot dismissed kevinbackhouse’s stale review December 29, 2022 16:17

Pull request has been modified.

@norbertwg
Copy link
Contributor Author

I updated my branch to reflect latest changes in main. Still one failed check:
Nightly - Linux distributions / distros (alpine:3.13, clang++, Release, OFF) (pull_request)
https://github.com/Exiv2/exiv2/actions/runs/3801758971/jobs/6466577419#logs
at installing dependencies

@kmilos
Copy link
Collaborator

kmilos commented Dec 29, 2022

I've restarted the job, let's see...

Btw, for future reference, please rebase the PR branch on top of main, not merge main into it.

@piponazo piponazo merged commit 9ca161d into Exiv2:main Dec 30, 2022
@norbertwg norbertwg deleted the rename-duplicate-exif-keys branch July 30, 2023 14:57
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.

4 participants