Skip to content

Fix build warnings from clang - #282

Merged
hainest merged 27 commits into
masterfrom
thaines/clang_cleanup
Oct 17, 2026
Merged

hainest merged 27 commits into
masterfrom
thaines/clang_cleanup

Conversation

@hainest

@hainest hainest commented Oct 16, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@hainest hainest self-assigned this Oct 16, 2026

@bbiiggppiigg bbiiggppiigg left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The resumeLog issue seems real.
The null check might be worth having.
The rest are probably fine?

Comment thread src/ResumeLog.C Resolved
Comment thread src/dyninst/dyninst_comp.C Resolved
Comment thread src/dyninst/dyninst_comp.C Resolved
Comment thread src/remotetest.C Resolved
Comment thread .github/workflows/pr-tests.yaml Resolved
@hainest
hainest requested a review from bbiiggppiigg October 17, 2026 02:28
Comment thread src/ResumeLog.C Resolved
Comment thread src/remotetest.C Resolved

@bbiiggppiigg bbiiggppiigg left a comment •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM then

@hainest
hainest merged commit c89cfcd into master Oct 17, 2026
7 checks passed
@hainest
hainest deleted the thaines/clang_cleanup branch October 17, 2026 21:26
hainest added a commit to dyninst/dyninst that referenced this pull request Nov 4, 2026
hainest added a commit to dyninst/dyninst that referenced this pull request Nov 8, 2026
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.

3 participants