code review 150730043: os: Emulate plan 9 libc in stat

os: Emulate plan 9 libc in stat

If the stat buffer is too short, the kernel informs us by putting the 2-byte
size in the buffer, so we read that and try again.

This follows the same algorithm as /sys/src/libc/9sys/dirfstat.c.

Fixes issue 8781.

[mischief, 2014-09-19 22:43:08]

Hello aram@mgk.ro, ality@pbrane.org, 0intro@gmail.com (cc:
golang-codereviews@googlegroups.com, rsc@golang.org),

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

[rsc, 2014-09-19 23:07:10]

please put _ at the beginning of the new constant names. the QIDSZ is maybe
a little useful but the others are not.

[mischief, 2014-09-19 23:22:00]

Hello aram@mgk.ro, ality@pbrane.org, 0intro@gmail.com (cc:
golang-codereviews@googlegroups.com, rsc@golang.org),

Please take another look.

[0intro, 2014-09-20 07:26:21]

I think you may want to move the _BIT8SZ constants to the os package.

[mischief, 2014-10-06 05:32:02]

Hello aram@mgk.ro, ality@pbrane.org, 0intro@gmail.com (cc:
golang-codereviews@googlegroups.com, rsc@golang.org),

Please take another look.

[0intro, 2014-10-13 19:40:58]

It looks good to me. Waiting for Russ for approval.

[gobot, 2014-12-19 05:13:44]

R=close

To the author of this CL:

The Go project has moved to Gerrit Code Review.

If this CL should be continued, please see the latest version of
https://golang.org/doc/contribute.html for instructions on
how to set up Git and the Go project's Gerrit codereview plugin,
and then create a new change with your current code.

If there has been discussion on this CL, please give a link to it
(golang.org/cl/150730043 is best) in the description in your
new CL.

Thanks very much.