Updated MakerNote capitalization by radarhere · Pull Request #8615 · python-pillow/Pillow
Navigation Menu
{{ message }}
python-pillow / Pillow Public
- Notifications You must be signed in to change notification settings
- Fork 2.4k
Merged
hugovk merged 2 commits intopython-pillow:mainfrom
Dec 21, 2024Merged
Updated MakerNote capitalization#8615
hugovk merged 2 commits intopython-pillow:mainfrom
Updated MakerNote capitalization#8615
hugovk merged 2 commits intopython-pillow:mainfrom
Conversation
Copy link Copy Markdown
Member
radarhere
commented
Dec 21, 2024
radarhere
commented
Resolves #8614
The issue has pointed out that the capitalization between
Line 156 in cf4110f
| MakerNote = 0x927C |
and
Line 355 in cf4110f
| Makernote = 37500 |
is inconsistent.
I've chosen MakerNote, to match https://www.loc.gov/preservation/digital/formats/content/tiff_tags.shtml
This also updates ExifTags to use hex values more consistently.
radarhere added 2 commits
December 22, 2024 07:24
radarhere
added
the
Exif
label
radarhere
mentioned this pull request
hugovk
merged commit
1146190
into
python-pillow:main
radarhere
deleted the
exiftags
branch
radarhere
mentioned this pull request
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment