Code Review for 6068-libdisasm-previnstr-arch-op-should-have-a-sane-default

Prepared by: (jeffpc) on 2015-Oct-14 16:45 -0400 EDT
Workspace:/home/jeffpc/illumos/mf-disasm (at e159b54f1d06)
Compare against: HEAD^ (at 5694aa7bc3b8)
Summary of changes: 115 lines changed: 58 ins; 57 del; 0 mod; 802 unchg
Patch of changes: 6068-libdisasm-previnstr-arch-op-should-have-a-sane-default.patch
Printable review: 6068-libdisasm-previnstr-arch-op-should-have-a-sane-default.pdf

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

6068 libdisasm: previnstr arch op should have a sane default
Reviewed by: Robert Mustacchi <rm@joyent.com>
52 lines changed: 0 ins; 52 del; 0 mod; 204 unchg

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

6068 libdisasm: previnstr arch op should have a sane default
Reviewed by: Robert Mustacchi <rm@joyent.com>
5 lines changed: 0 ins; 5 del; 0 mod; 354 unchg

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

6068 libdisasm: previnstr arch op should have a sane default
Reviewed by: Robert Mustacchi <rm@joyent.com>
58 lines changed: 58 ins; 0 del; 0 mod; 244 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.