[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] list_padding command
- To: Florian Bruhin <me _at_ the _minus_ compiler _dot_ org>
- Subject: Re: [PATCH] list_padding command
- From: Thorsten Wißmann <re06huxa _at_ cip _dot_ cs _dot_ fau _dot_ de>
- Date: Wed, 31 Oct 2012 16:17:56 +0100
Hi Florian,
On Wed, Oct 31, 2012 at 07:02:41AM +0100, Florian Bruhin wrote:
> Sorry if you need to copypaste the patch, don't have an MTA set up, so
> no idea how to use git send-email correcly.
Your mail was absolutely well-formatted. And the attached patch could be
easily piped into git-am. So it's totally OK.
> If anything is wrong or can be improved, let me know.
>
> Sample output:
>
> $ herbstclient list_padding
> 0: 24 0 0 0
> 1: 23 42 23 42
I think that's the most simple way of implementing it. So the patch
looks very canonical. I merged it as:
77539ca new command: list_padding
As soon there are monitor names we can add a parameter that directly
prints the padding of a certain monitor without the "INDEX:" prefix.
Thorsten