Skip to content

jpg: Fix silent IPTC loss after XMP segment - #66

Merged
bep merged 2 commits into
bep:mainfrom
LNA-DEV:lna/xmp-misalignment
Apr 16, 2026
Merged

jpg: Fix silent IPTC loss after XMP segment#66
bep merged 2 commits into
bep:mainfrom
LNA-DEV:lna/xmp-misalignment

Conversation

@LNA-DEV

@LNA-DEV LNA-DEV commented Apr 15, 2026

Copy link
Copy Markdown
Contributor

This adds a fix for #65

For more details take a look at the issue

Tests

I tested this fix with the unit tests and a custom hugo build from the master branch. Both worked fine after the fix.

@codecov-commenter

codecov-commenter commented Apr 15, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 71.42857% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 78.36%. Comparing base (f2f3657) to head (22477a6).

Files with missing lines Patch % Lines
imagedecoder_jpg.go 71.42% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #66      +/-   ##
==========================================
- Coverage   78.41%   78.36%   -0.05%     
==========================================
  Files          16       16              
  Lines        2103     2108       +5     
==========================================
+ Hits         1649     1652       +3     
- Misses        317      318       +1     
- Partials      137      138       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@bep
bep force-pushed the lna/xmp-misalignment branch from 1dd0a26 to d462fb3 Compare April 16, 2026 13:33
@bep
bep force-pushed the lna/xmp-misalignment branch from d462fb3 to 22477a6 Compare April 16, 2026 13:42
@bep
bep merged commit 8e8cc43 into bep:main Apr 16, 2026
6 checks passed
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.

3 participants