code review 89830045: net: extend TestVariousDeadlines1Proc timeout

net: extend TestVariousDeadlines1Proc timeout

TestVariousDeadlines1Proc was flaky on my system,
failing on about 5% of runs.

[josharian, 2014-04-21 20:00:02]

Hello golang-codereviews@googlegroups.com,

I'd like you to review this change to
https://code.google.com/p/go

[bradfitz, 2014-04-21 20:02:10]

LGTM
 On Apr 21, 2014 1:00 PM, <josharian@gmail.com> wrote:

> Reviewers: golang-codereviews,
>
> Message:
> Hello golang-codereviews@googlegroups.com,
>
> I'd like you to review this change to
> https://code.google.com/p/go
>
>
> Description:
> net: extend TestVariousDeadlines1Proc timeout
>
> TestVariousDeadlines1Proc was flaky on my system,
> failing on about 5% of runs.
>
> Please review this at https://codereview.appspot.com/89830045/
>
> Affected files (+1, -4 lines):
>   M src/pkg/net/timeout_test.go
>
>
> Index: src/pkg/net/timeout_test.go
> ===================================================================
> --- a/src/pkg/net/timeout_test.go
> +++ b/src/pkg/net/timeout_test.go
> @@ -494,10 +494,7 @@
>                                 clientc <- copyRes{n, err, d}
>                         }()
>
> -                       tooLong := 2 * time.Second
> -                       if runtime.GOOS == "windows" {
> -                               tooLong = 5 * time.Second
> -                       }
> +                       tooLong := 5 * time.Second
>                         select {
>                         case res := <-clientc:
>                                 if isTimeout(res.err) {
>
>
> --
> You received this message because you are subscribed to the Google Groups
> "golang-codereviews" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to golang-codereviews+unsubscribe@googlegroups.com.
> For more options, visit https://groups.google.com/d/optout.
>

[josharian, 2014-04-21 20:07:43]

*** Submitted as https://code.google.com/p/go/source/detail?r=c313673339c4 ***

net: extend TestVariousDeadlines1Proc timeout

TestVariousDeadlines1Proc was flaky on my system,
failing on about 5% of runs.

LGTM=bradfitz
R=golang-codereviews, bradfitz
CC=golang-codereviews
https://codereview.appspot.com/89830045