Skip to content

fix: keep saved markers in file order - #251

Open
mertalev wants to merge 1 commit into
google:mainfrom
mertalev:fix-marker-list-order
Open

mertalev wants to merge 1 commit into
google:mainfrom
mertalev:fix-marker-list-order

Conversation

@mertalev

@mertalev mertalev commented Oct 2, 2026

Copy link
Copy Markdown

Description

The current behavior of inserting new markers to the head of the list deviates from the libjpeg API, which documents that the list is "in order of their occurrence in the file". libultrahdr relies on this order to compute EXIF offset by summing the lengths of the saved markers that precede it.

Pull Request Checklist

  • CLA Signed: Have you signed the Contributor License Agreement (individual or corporate, as appropriate)? Only contributions from signed contributors can be accepted.
  • Authors: Have you considered adding your name to the AUTHORS file?
  • Code Style: Have you ensured your code adheres to the project's coding style guidelines? You can use ./ci.sh lint for automatic code formatting.

Please review the full contributing guidelines for more details.

This branch has not been deployed

No deployments
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.

1 participant