public inbox for developer@lists.illumos.org (since 2011-08)
 help / color / mirror / Atom feed
* Review - fmdump improvements: 11745, 16051-2, 16054, 16056
@ 2024-04-02 16:34 Keith M Wesolowski
  0 siblings, 0 replies; only message in thread
From: Keith M Wesolowski @ 2024-04-02 16:34 UTC (permalink / raw)
  To: illumos-developer

Please review my changes to fmdump and its documentation which may be found at
https://code.illumos.org/c/illumos-gate/+/3150.  The bugs and RFEs addressed
by this set of changes are:

11745 fmdump missing -i and -I parameter documentation in the man pages
16051 fmdump -A, -a, -H, -j, and -p should be documented
16052 fmdump -O uses unchecked strtoull
16054 fmdump mishandles interleaved arguments with -A
16056 want fmdump ability to AND event property filters

https://www.illumos.org/issues/11745
https://www.illumos.org/issues/16051
https://www.illumos.org/issues/16052
https://www.illumos.org/issues/16054
https://www.illumos.org/issues/16056

There are test results in each of the tickets except for 16051 which is
covered along with 11745.  There is also test data, which happens to be the
exact body of data that inspired the RFE in 16056, included with that ticket
if anyone wants to examine or extend it into the start of a test suite.  The
additions to the manual should help clarify the intent behind these changes if
the text in the tickets isn't enough.

Please let me know if there's anything else that would aid review.  These
changes aren't very large or complex despite the number of tickets and seeming
complexity; most of it is new block comments and doc updates.

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2024-04-02 16:35 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-04-02 16:34 Review - fmdump improvements: 11745, 16051-2, 16054, 16056 Keith M Wesolowski

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).