Page MenuHomeFreeBSD
Differential D33700

ctfconvert: Handle arrays of empty structs
ClosedPublic
Actions

Authored by markj on Dec 30 2021, 3:12 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Nov 30, 10:31 PM
Unknown Object (File)
Sat, Nov 16, 7:10 PM
Unknown Object (File)
Wed, Nov 13, 5:52 PM
Unknown Object (File)
Mon, Nov 11, 8:38 PM
Unknown Object (File)
Oct 28 2024, 10:45 PM
Unknown Object (File)
Oct 9 2024, 7:25 AM
Unknown Object (File)
Oct 1 2024, 1:19 PM
Unknown Object (File)
Sep 22 2024, 10:32 PM
View All 45 Files
Subscribers
bz
imp

Details

Reviewers
bz
Commits
rG963f5dc7a306: ctfconvert: Handle arrays of empty structs
Summary

Members with such a type will legitimately have a size of zero, so don't
emit a warning.

PR: 260818

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

markj created this revision.Dec 30 2021, 3:12 PM
Herald added a subscriber: imp. · View Herald TranscriptDec 30 2021, 3:12 PM
markj requested review of this revision.Dec 30 2021, 3:12 PM
Harbormaster completed remote builds in B43650: Diff 100755.Dec 30 2021, 3:12 PM
bz accepted this revision.Dec 30 2021, 5:16 PM
bz added a subscriber: bz.

Sounds good to me; maybe the comment above that block could be enhanced as well but I think it's almost immediately obvious from the code.

This revision is now accepted and ready to land.Dec 30 2021, 5:16 PM
Closed by commit rG963f5dc7a306: ctfconvert: Handle arrays of empty structs (authored by markj). · Explain WhyDec 31 2021, 6:09 PM
This revision was automatically updated to reflect the committed changes.
markj added a commit: rG963f5dc7a306: ctfconvert: Handle arrays of empty structs.

Revision Contents
Changeset List

PathSize
cddl/
contrib/
opensolaris/
tools/
ctf/
cvt/
12 lines

Diff 100791

View Options

cddl/contrib/opensolaris/tools/ctf/cvt/dwarf.c

Loading...