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
F132247390: D33700.id100755.diff
Wed, Oct 15, 4:11 AM
Unknown Object (File)
Sun, Oct 12, 11:19 PM
Unknown Object (File)
Sun, Oct 12, 1:52 PM
Unknown Object (File)
Sun, Oct 12, 7:11 AM
Unknown Object (File)
Sun, Oct 12, 7:10 AM
Unknown Object (File)
Wed, Oct 8, 7:07 PM
Unknown Object (File)
Wed, Oct 1, 4:14 AM
Unknown Object (File)
Sep 11 2025, 3:55 AM
View All 79 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...