[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: UPDATE: command: shift_edge



Hi Tylo,

On Sat, Dec 08, 2012 at 10:11:47PM -0800, Tyler Thomas Hart wrote:
> You have a valid point about 'shift' changing. Note, though, that
> cyclical-shifting (edge-wrapping) is the MAIN reason why I wrote shift-edge:
> 
> (shift right || shift_edge left)
> herbstclient keybind $foo or , shift -i right , shift_edge left

That's a cool and useful usage of shift_edge, I didn't think of that.

> Also, please note my response to a comment in your reply to 'query':
> >An alternative solution would be writing a command that takes any
> >other command, and repeats it (until a nonzero). This would take only
> >a few lines of code, and have greater scope and more uses than my
> >command.  But it would also contribute to my yet-another scripting
> >language paradigm, and it would also be DANGEROUS without safeguards
> >like a max_repeat or something similar. So your call.
> 
> bearing that in mind, (I, personally, think that adding a more general
> command would be a BAD idea), if you don't want to impliment the more
> general solution, I acknowledge your changes to my patch and wish for them
> to be commited to master.

Yes, something like repeat would be much more dangerous than
implementing the loop in herbstluftwm itself, so it's merged now:

0cf1c23 Add command shift_edge

Regards,
Thorsten