Code Review for illumos

Prepared by:Joshua M. Clulow (jclulow) on 2012-Nov-29 01:04 +0000 UTC
Workspace:/ws/plat/projects/illumos (at a1119fb89737)
Compare against: origin/master (git://github.com/joyent/illumos-joyent at ced3e3213686)
Summary of changes: 1415 lines changed: 1410 ins; 1 del; 4 mod; 20614 unchg
Patch of changes: illumos.patch

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/cmd/dtrace/test/tst/common/Makefile

OS-1723 DTrace should speak JSON
Reviewed by: Bryan Cantrill <bmc@joyent.com>
13 lines changed: 13 ins; 0 del; 0 mod; 125 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/cmd/dtrace/test/tst/common/aggs/tst.subr.d

OS-1723 DTrace should speak JSON
Reviewed by: Bryan Cantrill <bmc@joyent.com>
3 lines changed: 3 ins; 0 del; 0 mod; 113 unchg

------ ------ ------ ------ ------ --- New Patch Raw usr/src/cmd/dtrace/test/tst/common/json/tst.general.d

OS-1723 DTrace should speak JSON
Reviewed by: Bryan Cantrill <bmc@joyent.com>
179 lines changed: 179 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ------ --- New Patch Raw usr/src/cmd/dtrace/test/tst/common/json/tst.general.d.out

OS-1723 DTrace should speak JSON
Reviewed by: Bryan Cantrill <bmc@joyent.com>
218 lines changed: 218 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ------ --- New Patch Raw usr/src/cmd/dtrace/test/tst/common/json/tst.strsize.d

OS-1723 DTrace should speak JSON
Reviewed by: Bryan Cantrill <bmc@joyent.com>
51 lines changed: 51 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ------ --- New Patch Raw usr/src/cmd/dtrace/test/tst/common/json/tst.strsize.d.out

OS-1723 DTrace should speak JSON
Reviewed by: Bryan Cantrill <bmc@joyent.com>
13 lines changed: 13 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ------ --- New Patch Raw usr/src/cmd/dtrace/test/tst/common/json/tst.usdt.c

OS-1723 DTrace should speak JSON
Reviewed by: Bryan Cantrill <bmc@joyent.com>
61 lines changed: 61 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ------ --- New Patch Raw usr/src/cmd/dtrace/test/tst/common/json/tst.usdt.d

OS-1723 DTrace should speak JSON
Reviewed by: Bryan Cantrill <bmc@joyent.com>
65 lines changed: 65 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ------ --- New Patch Raw usr/src/cmd/dtrace/test/tst/common/json/tst.usdt.d.out

OS-1723 DTrace should speak JSON
Reviewed by: Bryan Cantrill <bmc@joyent.com>
11 lines changed: 11 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ------ --- New Patch Raw usr/src/cmd/dtrace/test/tst/common/json/usdt.d

OS-1723 DTrace should speak JSON
Reviewed by: Bryan Cantrill <bmc@joyent.com>
27 lines changed: 27 ins; 0 del; 0 mod; 0 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/cmd/dtrace/test/tst/common/privs/tst.func_access.ksh

OS-1723 DTrace should speak JSON
Reviewed by: Bryan Cantrill <bmc@joyent.com>
8 lines changed: 6 ins; 1 del; 1 mod; 80 unchg

------ ------ ------ ------ ------ --- New Patch Raw usr/src/cmd/dtrace/test/tst/common/strtoll/err.BaseTooLarge.d

OS-1723 DTrace should speak JSON
Reviewed by: Bryan Cantrill <bmc@joyent.com>
35 lines changed: 35 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ------ --- New Patch Raw usr/src/cmd/dtrace/test/tst/common/strtoll/err.BaseTooSmall.d

OS-1723 DTrace should speak JSON
Reviewed by: Bryan Cantrill <bmc@joyent.com>
34 lines changed: 34 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ------ --- New Patch Raw usr/src/cmd/dtrace/test/tst/common/strtoll/tst.strtoll.d

OS-1723 DTrace should speak JSON
Reviewed by: Bryan Cantrill <bmc@joyent.com>
66 lines changed: 66 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ------ --- New Patch Raw usr/src/cmd/dtrace/test/tst/common/strtoll/tst.strtoll.d.out

OS-1723 DTrace should speak JSON
Reviewed by: Bryan Cantrill <bmc@joyent.com>
20 lines changed: 20 ins; 0 del; 0 mod; 0 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/common/util/strtolctype.h

OS-1723 DTrace should speak JSON
Reviewed by: Bryan Cantrill <bmc@joyent.com>
4 lines changed: 4 ins; 0 del; 0 mod; 75 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/lib/libdtrace/common/dt_open.c

OS-1723 DTrace should speak JSON
Reviewed by: Bryan Cantrill <bmc@joyent.com>
8 lines changed: 6 ins; 0 del; 2 mod; 1411 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/uts/common/dtrace/dtrace.c

OS-1723 DTrace should speak JSON
Reviewed by: Bryan Cantrill <bmc@joyent.com>
593 lines changed: 593 ins; 0 del; 0 mod; 16296 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/uts/common/sys/dtrace.h

OS-1723 DTrace should speak JSON
Reviewed by: Bryan Cantrill <bmc@joyent.com>
3 lines changed: 2 ins; 0 del; 1 mod; 2338 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/uts/intel/dtrace/Makefile

OS-1723 DTrace should speak JSON
Reviewed by: Bryan Cantrill <bmc@joyent.com>
2 lines changed: 2 ins; 0 del; 0 mod; 79 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/uts/sparc/dtrace/Makefile

OS-1723 DTrace should speak JSON
Reviewed by: Bryan Cantrill <bmc@joyent.com>
1 line changed: 1 ins; 0 del; 0 mod; 97 unchg

This code review page was prepared using /ws/plat/projects/illumos/usr/src/tools/proto/root_i386-nd/opt/onbld/bin/webrev. Webrev is maintained by the illumos project. The latest version may be obtained here.