code review 163050043: spec: minimal documention of unsafe.Pointer conversions

spec: minimal documention of unsafe.Pointer conversions

Per suggestion from rsc as a result of the dicussion of
(abandoned) CL 153110044.

Fixes issue 7192.

[gri, 2014-10-23 00:28:21]

Hello r@golang.org, rsc@golang.org, iant@golang.org, ken@golang.org (cc:
golang-codereviews@googlegroups.com),

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

[r, 2014-10-23 00:49:45]

LGTM

https://codereview.appspot.com/163050043/diff/40001/doc/go_spec.html
File doc/go_spec.html (right):

https://codereview.appspot.com/163050043/diff/40001/doc/go_spec.html#newcode6210
doc/go_spec.html:6210: must be vetted manually for type safety and may not be
portable.
i wonder if we should add a clause to ref/go1compat and/or pkg/unsafe

[rsc, 2014-10-23 03:56:30]

LGTM
the other places sound good to change too

[iant, 2014-10-23 14:53:28]

LGTM

[r, 2014-10-23 16:00:21]

https://codereview.appspot.com/162060043

On Wed, Oct 22, 2014 at 8:56 PM, <rsc@golang.org> wrote:

> LGTM
> the other places sound good to change too
>
>
> https://codereview.appspot.com/163050043/
>

[gri, 2014-10-23 16:45:17]

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

spec: minimal documention of unsafe.Pointer conversions

Per suggestion from rsc as a result of the dicussion of
(abandoned) CL 153110044.

Fixes issue 7192.

LGTM=r, rsc, iant
R=r, rsc, iant, ken
CC=golang-codereviews
https://codereview.appspot.com/163050043

[gobot, 2014-10-23 16:58:14]

This CL appears to have broken the plan9-amd64-aram builder.
See http://build.golang.org/log/351d6ff6c843928cd3fe92e2c512a811a54282fa