Skip to content

Conversation

@trumank
Copy link
Contributor

@trumank trumank commented Dec 23, 2025

I started digging into some vtable issues and there seem to be quite a few beyond just these trivial ones.

  • First fix is making QualifiedName default to :: separator like docs say it should which fixes VTable struct being incorrectly defined without :: e.g. MultiDerivedVTable
  • Second fix is in parsing vtable data symbol with explicit base class: e.g. MultiDerived::`vftable'{for `InterfaceB'} was getting incorrectly parsed as vftable'{for `InterfaceB::VTable

Test binaries: pdb-vtable.zip

Before:
Before fix

After:
After fix

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@emesare emesare self-assigned this Dec 23, 2025
@emesare emesare added DebugInfo: PDB Related to parsing PDB files Component: Rust API Issue needs changes to the Rust API labels Dec 23, 2025
@emesare emesare added this to the Jotunheim milestone Dec 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Component: Rust API Issue needs changes to the Rust API DebugInfo: PDB Related to parsing PDB files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants