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)
Mon, Feb 10, 9:00 PM
Unknown Object (File)
Mon, Feb 10, 8:58 PM
Unknown Object (File)
Wed, Feb 5, 6:02 AM
Unknown Object (File)
Fri, Jan 31, 11:25 PM
Unknown Object (File)
Fri, Jan 24, 11:51 AM
Unknown Object (File)
Mon, Jan 20, 4:05 PM
Unknown Object (File)
Sat, Jan 18, 12:31 AM
Unknown Object (File)
Jan 9 2025, 10:33 PM
View All 71 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...