On Fri, Mar 25, 2011 at 09:44:33AM +0000, Glen Anderson wrote: > > "combined" is ok with me, it doesn't imply anything about how the > figures are reached so side-steps the issues I had earlier. In the > esoteric case I mentioned people will make their own (probably > incorrect) assumptions with either wording. I've included a new diff > making it clear that they're combined statistics. > > Ultimately I think there should be a note explaining what happens in > the case of non-identical CPUs but without knowing what top does I'm > happy with either of the following diffs. > > I'll have a dig through the source to see if I can find out. I'm not > holding out much hope as "learn C" has been gathering dust on my todo > list for a while... A bit of code reading won't do me any harm though. > > $ diff -u top.1 top.1.new > --- top.1 Thu Mar 24 12:39:45 2011 > +++ top.1.new Fri Mar 25 09:28:16 2011 > @@ -75,7 +75,8 @@ > The options are as follows: > .Bl -tag -width Ds > .It Fl 1 > -Display CPU statistics on a single line instead of a line per CPU. > +Display combined CPU statistics on a single line instead of individual CPU > +statistics on multiple lines. > .It Fl b > Use > .Em batch > @@ -282,7 +283,8 @@ > .Sq P > interactive command. > .It 1 > -Display CPU statistics on a single line instead of a line per CPU. > +Toggle between combined CPU statistics on a single line and individual CPU > +statistics on multiple lines. > .It C > Toggle the display of process command line arguments. > .It d Ar count >
ok, i committed a diff, but a little bit different to the one above. funny how the simple diffs can make your head hurt most. in this case i'm not sure i like the idea of toggling "between" states. i think you toggle something on or off. god knows though. anyway i settled on the diff below once i got fed up staring at the screen. jmc Index: top.1 =================================================================== RCS file: /cvs/src/usr.bin/top/top.1,v retrieving revision 1.57 diff -u -r1.57 top.1 --- top.1 10 Aug 2010 20:34:16 -0000 1.57 +++ top.1 25 Mar 2011 12:52:58 -0000 @@ -75,7 +75,8 @@ The options are as follows: .Bl -tag -width Ds .It Fl 1 -Display CPU statistics on a single line instead of a line per CPU. +Display combined CPU statistics for all processors on a single line +instead of one line per CPU. .It Fl b Use .Em batch @@ -282,7 +283,7 @@ .Sq P interactive command. .It 1 -Display CPU statistics on a single line instead of a line per CPU. +Toggle the display of CPU statistics. .It C Toggle the display of process command line arguments. .It d Ar count