code review 117330044: runtime: fix freebsd build. Assignment is int32* on 32 bit

runtime: fix freebsd build.  Assignment is int32* on 32 bit
machines and int64* on 64 bit machines.

[khr, 2014-07-30 16:13:45]

Hello golang-codereviews@googlegroups.com,

I'd like you to review this change to
https://khr%40golang.org@code.google.com/p/go/

[bradfitz, 2014-07-30 16:16:35]

LGTM
 On Jul 30, 2014 9:13 AM, <khr@golang.org> wrote:

> Reviewers: golang-codereviews,
>
> Message:
> Hello golang-codereviews@googlegroups.com,
>
> I'd like you to review this change to
> https://khr%40golang.org@code.google.com/p/go/
>
>
> Description:
> runtime: fix freebsd build.  Assignment is int32* on 32 bit
> machines and int64* on 64 bit machines.
>
> Please review this at https://codereview.appspot.com/117330044/
>
> Affected files (+1, -1 lines):
>   M src/pkg/runtime/os_freebsd.c
>
>
> Index: src/pkg/runtime/os_freebsd.c
> ===================================================================
> --- a/src/pkg/runtime/os_freebsd.c
> +++ b/src/pkg/runtime/os_freebsd.c
> @@ -110,7 +110,7 @@
>         param.stack_base = (void*)mp->g0->stackbase;
>         param.stack_size = (byte*)stk - (byte*)mp->g0->stackbase;
>
> -       param.child_tid = (intptr*)&mp->procid;
> +       param.child_tid = (void*)&mp->procid;
>         param.parent_tid = nil;
>         param.tls_base = (void*)&mp->tls[0];
>         param.tls_size = sizeof mp->tls;
>
>
> --
> 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.
>

[khr, 2014-07-30 16:18:03]

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

runtime: fix freebsd build.  Assignment is int32* on 32 bit
machines and int64* on 64 bit machines.

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

[gobot, 2014-07-30 16:43:18]

This CL appears to have broken the darwin-386-cheney builder.
See http://build.golang.org/log/3a24bdbbf99be17f710ee548377c73282da612ef

[dave_cheney.net, 2014-07-30 21:55:30]

Not really,

ok  	cmd/fix	0.032s
--- FAIL: TestRepoRootForImportPath (95.75s)
	vcs_test.go:114: RepoRootForImport("code.google.com/p/go"): Get
https://code.google.com/p/go/source/checkout?repo=: dial tcp
150.101.161.152:443: i/o timeout
FAIL
FAIL	cmd/go	95.877s



On Thu, Jul 31, 2014 at 2:43 AM, <gobot@golang.org> wrote:

> This CL appears to have broken the darwin-386-cheney builder.
> See http://build.golang.org/log/3a24bdbbf99be17f710ee548377c73282da612ef
>
>
> https://codereview.appspot.com/117330044/
>
> --
> 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.
>