Page MenuHomeFreeBSD
Differential D29381

Drop WARNS for ktrdump(8)
ClosedPublic
Actions

Authored by dchagin on Mar 22 2021, 6:26 PM.
Tags
None
Referenced Files
F146593790: D29381.id86163.diff
Tue, Mar 3, 10:50 PM
F146579648: D29381.id.diff
Tue, Mar 3, 7:53 PM
F146532299: D29381.id.diff
Tue, Mar 3, 10:41 AM
Unknown Object (File)
Tue, Mar 3, 6:56 AM
Unknown Object (File)
Jan 5 2026, 6:41 PM
Unknown Object (File)
Dec 10 2025, 12:42 PM
Unknown Object (File)
Dec 10 2025, 12:41 PM
Unknown Object (File)
Dec 5 2025, 1:22 PM
View All Files
Subscribers
imp

Details

Reviewers
jhb
imp
Commits
rGb4dc951c417e: Fix warning about signed comparison and drop WARNS for ktrdump(8).
rG3802433d25c9: Fix warning about signed comparison and drop WARNS for ktrdump(8).
rG9e5aeba51b43: Fix warning about signed comparison and drop WARNS for ktrdump(8).

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 37995
Build 34884: arc lint + arc unit

Event Timeline

dchagin created this revision.Mar 22 2021, 6:26 PM
Herald added a subscriber: imp. · View Herald TranscriptMar 22 2021, 6:26 PM
dchagin requested review of this revision.Mar 22 2021, 6:26 PM
Harbormaster completed remote builds in B37995: Diff 86163.Mar 22 2021, 6:26 PM
dchagin added a reviewer: jhb.Mar 22 2021, 6:27 PM
jhb added inline comments.Mar 22 2021, 7:19 PM
usr.bin/ktrdump/ktrdump.c
59

Could we avoid all the 0's if we used this instead:

{ .n_name = "_ktr_version" },

If so, I think I'd prefer that style.

dchagin updated this revision to Diff 86164.Mar 22 2021, 7:37 PM

Sure, done

Harbormaster completed remote builds in B37996: Diff 86164.Mar 22 2021, 7:38 PM
jhb accepted this revision.Mar 23 2021, 7:45 PM
This revision is now accepted and ready to land.Mar 23 2021, 7:45 PM
imp accepted this revision.Mar 23 2021, 7:51 PM
Closed by commit rG9e5aeba51b43: Fix warning about signed comparison and drop WARNS for ktrdump(8). (authored by dchagin). · Explain WhyMar 24 2021, 7:53 AM
This revision was automatically updated to reflect the committed changes.
dchagin added a commit: rG9e5aeba51b43: Fix warning about signed comparison and drop WARNS for ktrdump(8)..
dchagin added a commit: rG3802433d25c9: Fix warning about signed comparison and drop WARNS for ktrdump(8)..Apr 1 2021, 7:59 AM
dchagin added a commit: rGb4dc951c417e: Fix warning about signed comparison and drop WARNS for ktrdump(8)..Apr 1 2021, 8:02 AM

Revision Contents
Changeset List

PathSize
usr.bin/
ktrdump/
2 lines
14 lines

Diff 86163

View Options

usr.bin/ktrdump/Makefile

Loading...
View Options

usr.bin/ktrdump/ktrdump.c

Loading...