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
Unknown Object (File)
Wed, Dec 10, 12:42 PM
Unknown Object (File)
Wed, Dec 10, 12:41 PM
Unknown Object (File)
Fri, Dec 5, 1:22 PM
Unknown Object (File)
Fri, Nov 28, 5:44 AM
Unknown Object (File)
Thu, Nov 27, 12:59 PM
Unknown Object (File)
Nov 13 2025, 8:27 AM
Unknown Object (File)
Nov 13 2025, 7:51 AM
Unknown Object (File)
Nov 13 2025, 7:41 AM
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 37996
Build 34885: 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–63

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 86164

View Options

usr.bin/ktrdump/Makefile

Loading...
View Options

usr.bin/ktrdump/ktrdump.c

Loading...