Re: Monitor silence patch.

2010-12-06 Thread Thomas Adam
On Mon, Dec 06, 2010 at 10:05:21PM +, Nicholas Marriott wrote: > Basically newline after every period. OK. I *hope* I've cleaned that up enough. Patch attached. > It means that you don't end up with a gigantic diff reflowing entire > paragraphs if someone changes one word in a sentence. Ma

Re: Monitor silence patch.

2010-12-06 Thread Nicholas Marriott
On Mon, Dec 06, 2010 at 09:50:23PM +, Thomas Adam wrote: > Hi -- > > On Mon, Dec 06, 2010 at 09:39:33PM +, Nicholas Marriott wrote: > > I like it and looks fine apart from a couple of man page nits: > > OK. I am sure I can iron those out. ;) > > > - Each new sentence gets a new line (

Re: Monitor silence patch.

2010-12-06 Thread Thomas Adam
Hi -- On Mon, Dec 06, 2010 at 09:39:33PM +, Nicholas Marriott wrote: > I like it and looks fine apart from a couple of man page nits: OK. I am sure I can iron those out. ;) > - Each new sentence gets a new line (reduces diff size for minor > fixes). I don't understand what you mean here.

Re: Monitor silence patch.

2010-12-06 Thread Nicholas Marriott
I like it and looks fine apart from a couple of man page nits: - Each new sentence gets a new line (reduces diff size for minor fixes). - We document what * and + etc mean (look for "Denotes") so you need to add ~. It's dependent on another event waking us up, but that's fine because the second

Monitor silence patch.

2010-12-06 Thread Thomas Adam
Hi all, Please see the patch attahed which should implement "monitor-silence" and other options, along the same lines as the monitor-activity options. I've also included man page updates. Feedback welcome. Any questions, just shout. The patch is applied on top of the sourceforge CVS repository