Code Review for illumos-gate

Prepared by:david (david) on 2013-Jan-24 11:08 +0000 UTC
Workspace:/home/david/ws/illumos-gate (at 43d73c5f435c)
Compare against: b7c47e8c412d4654019e45625cf2be8f7fd6ee66 (at b7c47e8c412d)
Summary of changes: 2122 lines changed: 1818 ins; 294 del; 10 mod; 408 unchg
Patch of changes: illumos-gate.patch
Printable review: illumos-gate.pdf

------ ------ ------ ------ ------ Old --- Patch --- usr/src/cmd/kstat/Makefile (deleted)

749 "/usr/bin/kstat" should be rewritten in C
Reviewed by: Garrett D'Amore <garrett@damore.org>
Reviewed by: Brendan Gregg <brendan.gregg@joyent.com>
47 lines changed: 0 ins; 47 del; 0 mod; 0 unchg

------ ------ ------ ------ ------ Old --- Patch --- usr/src/cmd/kstat/kstat.pl (deleted)

749 "/usr/bin/kstat" should be rewritten in C
Reviewed by: Garrett D'Amore <garrett@damore.org>
Reviewed by: Brendan Gregg <brendan.gregg@joyent.com>
245 lines changed: 0 ins; 245 del; 0 mod; 0 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/cmd/stat/Makefile

749 "/usr/bin/kstat" should be rewritten in C
Reviewed by: Garrett D'Amore <garrett@damore.org>
Reviewed by: Brendan Gregg <brendan.gregg@joyent.com>
3 lines changed: 0 ins; 2 del; 1 mod; 45 unchg

------ ------ ------ ------ ------ --- New Patch Raw usr/src/cmd/stat/kstat/Makefile

749 "/usr/bin/kstat" should be rewritten in C
Reviewed by: Garrett D'Amore <garrett@damore.org>
Reviewed by: Brendan Gregg <brendan.gregg@joyent.com>
61 lines changed: 61 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ------ --- New Patch Raw usr/src/cmd/stat/kstat/kstat.c

749 "/usr/bin/kstat" should be rewritten in C
Reviewed by: Garrett D'Amore <garrett@damore.org>
Reviewed by: Brendan Gregg <brendan.gregg@joyent.com>
1470 lines changed: 1470 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ------ --- New Patch Raw usr/src/cmd/stat/kstat/kstat.h

749 "/usr/bin/kstat" should be rewritten in C
Reviewed by: Garrett D'Amore <garrett@damore.org>
Reviewed by: Brendan Gregg <brendan.gregg@joyent.com>
269 lines changed: 269 ins; 0 del; 0 mod; 0 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/man/man1m/kstat.1m

749 "/usr/bin/kstat" should be rewritten in C
Reviewed by: Garrett D'Amore <garrett@damore.org>
Reviewed by: Brendan Gregg <brendan.gregg@joyent.com>
27 lines changed: 18 ins; 0 del; 9 mod; 363 unchg

This code review page was prepared using /opt/onbld/bin/webrev. Webrev is maintained by the illumos project. The latest version may be obtained here.