code review 223108: net: fix network timeout boundary condition.

net: fix network timeout boundary condition.
Fixes issue 472.

[stephenm, 2010-03-03 06:56:43]

Hello golang-dev@googlegroups.com (cc: golang-dev@googlegroups.com),

I'd like you to review this change.

[rsc, 2010-03-03 07:01:39]

LGTM

Thanks.

Before submitting, run "hg change" to edit the description to say
Fixes instead of Fixed, so that the google code commit hooks
will auto-close the issue.



On Tue, Mar 2, 2010 at 22:56, <stephenm@golang.org> wrote:

> Reviewers: golang-dev_googlegroups.com,
>
> Message:
> Hello golang-dev@googlegroups.com (cc: golang-dev@googlegroups.com),
>
> I'd like you to review this change.
>
>
> Description:
> net: fix network timeout boundary condition.
> Fixed issue 472.
>
> Please review this at http://codereview.appspot.com/223108/show
>
> Affected files:
>  M src/pkg/net/fd.go
>
>
> Index: src/pkg/net/fd.go
> ===================================================================
> --- a/src/pkg/net/fd.go
> +++ b/src/pkg/net/fd.go
> @@ -228,7 +228,7 @@
>                var t = s.deadline
>                if t > 0 {
>                        t = t - s.Now()
> -                       if t < 0 {
> +                       if t <= 0 {
>                                s.CheckDeadlines()
>                                continue
>                        }
>
>
>

[stephenm, 2010-03-03 07:43:43]

*** Submitted as 55fce9d5068a ***

net: fix network timeout boundary condition.
Fixes issue 472.

R=golang-dev
CC=golang-dev
http://codereview.appspot.com/223108