On 27.11.2012 23:48, Robert Watson wrote:
On Tue, 27 Nov 2012, Andre Oppermann wrote:
On 27.11.2012 23:35, Peter Wemm wrote:
Andre.. this breaks incoming connections. TCP is immediately reset and never
even gets to the
listener process. You need to back out of fix this urgently please.
I just found out and fixed it. Sorry for the breakage.
I'd like to see a much more thorough use of "Reviewed by:" in socket and
TCP-related commits -- this
is very sensitive code, and a second pair of eyes is always valuable.
Post-commit review is not a
substitute. Looking back over similar changes in the socket code over the last
two years, I see
that almost all have reviewers, so I think it would be reasonable to consider
it mandatory for these
subsystems at this point. The good news is that we have lots of people with
expertise in it.
Good to see you becoming more active again. :-) And yes,
you have a point there.
--
Andre
_______________________________________________
svn-src-head@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"
