On 10/25/11 22:26, Rich Burridge wrote:
Hi,
Could I please get a code review for my fix for:
6489402 miscellaneous manpage format bugs: bash(1)
http://monaco.us.oracle.com/detail.jsf?cr=6489402
Webrev is at:
http://jurassic.us.oracle.com/~richb/6489402-v1
"diff -u" is preferred over "diff -c" for patches in general, and
would match the existing format, which would have made your webrev
smaller and less confusing. (It also looks like the previous one
was gnu diff, given the much more detailed timestamps than Solaris
diff generates).
--
-Alan Coopersmith- [email protected]
Oracle Solaris Platform Engineering: X Window System
_______________________________________________
userland-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/userland-discuss
