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

feat: log stacktrace implementation used #1769

Merged
merged 4 commits into from
Jun 22, 2024
Merged

Conversation

iTrooz
Copy link
Collaborator

@iTrooz iTrooz commented Jun 22, 2024

Related to #1768

This pull request introduces changes to enhance the logging functionality in ImHex by displaying the stacktrace implementation used when showing a stack trace.

  • Stacktrace Implementation Logging: Modifies getStackTrace() in main/gui/source/stacktrace.cpp to return a StackTraceResult struct, which includes both the stack frames and the name of the stacktrace implementation used (Windows, execinfo, or backtrace depending on the compilation flags). This allows for the identification of the stacktrace implementation in the logs.
  • Crash Handler Update: Updates printStackTrace() in main/gui/source/crash_handlers.cpp to log the name of the stacktrace implementation used before printing the stack frames. This ensures that the implementation name is clearly logged whenever a stack trace is printed due to a crash.

For more details, open the Copilot Workspace session.

@iTrooz iTrooz force-pushed the iTrooz/log-stacktrace-impl branch from 0107c52 to 011ad36 Compare June 22, 2024 16:28
@iTrooz iTrooz changed the title Log stacktrace implementation used feat: Log stacktrace implementation used Jun 22, 2024
@iTrooz iTrooz changed the title feat: Log stacktrace implementation used feat: log stacktrace implementation used Jun 22, 2024
@iTrooz iTrooz merged commit 65dfd4d into master Jun 22, 2024
17 checks passed
@iTrooz iTrooz deleted the iTrooz/log-stacktrace-impl branch June 22, 2024 19:33
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.

None yet

1 participant