Skip to content

Deduplicate SCIP symbol information - #482

Open
eseliger wants to merge 2 commits into
prototype-membersfrom
deduplicate-symbol-information
Open

eseliger wants to merge 2 commits into
prototype-membersfrom
deduplicate-symbol-information

Conversation

@eseliger

@eseliger eseliger commented Aug 22, 2026

Copy link
Copy Markdown
Member

Emit one SymbolInformation record per SCIP symbol. Overload declarations merge their unique documentation and relationships—including definition relationships—into the first record instead of emitting duplicate metadata.

The overload snapshot renders both overload signatures beneath one symbol, providing a focused regression test for the merge behavior.

This was reported by scip lint on a large repo.

@eseliger eseliger changed the title deduplicate symbol information Deduplicate SCIP symbol information Aug 22, 2026
@eseliger
eseliger force-pushed the deduplicate-symbol-information branch from af77128 to b6997da Compare August 23, 2026 00:08
@eseliger
eseliger marked this pull request as ready for review August 23, 2026 00:23
@christoph-sg
christoph-sg force-pushed the deduplicate-symbol-information branch from b6997da to d1877d4 Compare September 10, 2026 13:33
@christoph-sg
christoph-sg force-pushed the deduplicate-symbol-information branch from d1877d4 to 5b644a6 Compare September 11, 2026 08:58
@christoph-sg
christoph-sg force-pushed the deduplicate-symbol-information branch from 5b644a6 to 722f92e Compare September 11, 2026 09:03
@christoph-sg
christoph-sg force-pushed the deduplicate-symbol-information branch from 722f92e to 702fa82 Compare September 11, 2026 09:16
@christoph-sg
christoph-sg force-pushed the deduplicate-symbol-information branch from 702fa82 to f17bea7 Compare September 11, 2026 09:27
@christoph-sg
christoph-sg force-pushed the deduplicate-symbol-information branch from f17bea7 to 5b4ea78 Compare September 11, 2026 09:44
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