Thanks for adding this. A few tiny issues. http://codereview.appspot.com/96084/diff/1/3 File src/cmd/auxstats/Linux.c (right): http://codereview.appspot.com/96084/diff/1/3#newcode93 Line 93: ...
15 years, 5 months ago
(2009-07-25 18:35:51 UTC)
#2
Nope, there was no compiler warning, but that's understandable since sfree was used in only ...
15 years, 5 months ago
(2009-07-25 20:50:29 UTC)
#3
Nope, there was no compiler warning, but that's understandable since sfree was
used in only one if-block.
On 2009/07/25 18:35:51, rsc wrote:
> Thanks for adding this.
> A few tiny issues.
>
> http://codereview.appspot.com/96084/diff/1/3
> File src/cmd/auxstats/Linux.c (right):
>
> http://codereview.appspot.com/96084/diff/1/3#newcode93
> Line 93: mtot = stot = 0;
> move stot = 0 onto its own line
> add sfree = 0 too (wasn't there a compiler warning about this?)
Issue 96084: stats: make more stats work in Linux
(Closed)
Created 15 years, 5 months ago by fhs
Modified 15 years, 5 months ago
Reviewers:
Base URL:
Comments: 1