- At revision r247852 @mm accidentally we forgot to print out the obj too.
- Fix some void function that don't really need to return.
- Also for some global variables, declare it as static.
Details
Diff Detail
- Repository
- rS FreeBSD src repository - subversion
- Lint
No Lint Coverage - Unit
No Test Coverage
Event Timeline
Please note that the two comments inline, I think they are wrong and should be reverted. The rest part of the proposed patch looks good to me.
cddl/contrib/opensolaris/cmd/zdb/zdb.c | ||
---|---|---|
182 | This is wrong. usage() is supposed to be an exit point. | |
201 | This is also wrong, similar to usage() fatal is supposed to be a __dead exit. |
As @delphij pointed out, I have reverted those changes related with the 'exit(1);' case.
Adding Matt to reviewer so he gets notification. I think we should upstream most of the changes to Illumos -- Marcelo, can you create an Illumos ticket for this? I can create a review request there too but since you wrote the patch it's better that you get involved too :)
cddl/contrib/opensolaris/cmd/zdb/zdb.c | ||
---|---|---|
96 | These changes should be upstreamed to Illumos. | |
1489 | This should be upstreamed to Illumos. | |
1494 | I'm not sure about the style here -- I think if I was the author I would have a blank line on after line 1490, but it's not clear if it also matches Illumos code style. | |
1500 | This should be upstreamed too. |
@mahrens thanks for the review, yes, that supposed to be static too.
Please check this new version of the patch.
Best,
@delphij I had already open a BUG at Illumos project, here is the link: https://www.illumos.org/issues/5999
Thanks,