Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Embed PDB in DLL (for more descriptive errors) #82

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

extraes
Copy link
Contributor

@extraes extraes commented Aug 13, 2024

Makes VS embed the PDB into the DLL for both release and debug configurations.
I saw this option for Chaos which really fucking needed it and figured BoneLib might be able to get some use out of it.
The only downsides are as follows:

  • Probably slower exceptions (because the runtime would have to look up the line numbers)
  • Builder's directory structure (incl. user account name, if built under there, like in the default "repos" folder) being known (not a problem if using Actions to build, but idk if BoneLib does that)
  • Larger filesize (by probably not a lot)
  • ermmmm... too useful? idk.

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