Skip to content

COFF: Fix symbols being incorrectly sorted within section groups - #395

Merged
LagoLunatic merged 1 commit into
encounter:mainfrom
LagoLunatic:order-coff-groups
Aug 11, 2026
Merged

COFF: Fix symbols being incorrectly sorted within section groups#395
LagoLunatic merged 1 commit into
encounter:mainfrom
LagoLunatic:order-coff-groups

Conversation

@LagoLunatic

Copy link
Copy Markdown
Collaborator

objdiff sorts all symbols when reading the object (#316), but this currently only sorts them within their own section. COFF has groups of sections like rdata and multiple rdata$rs. This PR fixes the sorting so the symbols are sorted relative to other sections within the same group first. This fixes two bugs:

  • The combine sections feature would change the order of symbols when enabled (fixes Section merging seems to alter section order #392)
  • The order diffing feature incorrectly marks some symbols in grouped sections as being in the wrong order, even when they're in the right order and sections are unmerged
image

@tomsons26

Copy link
Copy Markdown
Contributor

Thanks
Works as expected now

@LagoLunatic
LagoLunatic merged commit 6735137 into encounter:main Aug 11, 2026
23 checks passed
@LagoLunatic
LagoLunatic deleted the order-coff-groups branch August 11, 2026 17:13
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.

Section merging seems to alter section order

2 participants