[PATCH 5/6] Show how VT100 ACS (alternate character set) should actually render

2012-09-16 Thread Marcel Partap
--- screen-redraw.c |4 ++-- tty-acs.c | 64 +++ 2 files changed, 34 insertions(+), 34 deletions(-) diff --git a/screen-redraw.c b/screen-redraw.c index 75f7238..7ffad30 100644 --- a/screen-redraw.c +++ b/screen-redraw.c @@ -41,8 +41

[PATCH 6/6] Implement mouse wheel scrolling emulation.

2012-09-16 Thread Marcel Partap
Scrolling mouse wheel while shift modifier key is held generates fake cursor key events. With ctrl modifier added, speed is multi- plied by the value of "mouse-scroll-speedup" variable (default 3). --- examples/tmux.vim |2 +- input-keys.c | 18 -- options-table.c |

[PATCH 4/6] Add pane-active-border-mark option for arrows <>^v indicating current pane.

2012-09-16 Thread Marcel Partap
--- examples/tmux.vim |2 +- options-table.c |5 + screen-redraw.c | 38 +++--- tmux.1|4 tty-term.c|2 +- 5 files changed, 38 insertions(+), 13 deletions(-) diff --git a/examples/tmux.vim b/examples/tmux.vim index

[PATCH 1/6] Find working path of foreground sub process in osdep_get_cwd

2012-09-16 Thread Marcel Partap
by picking up the process group leader from the terminal fd via tcgetpgrp() instead of using the direct pane child process. This correctly picks up the working directory for nested subshells. (works on linux, other platforms untested) --- cmd.c |2 +- format.c |2 +-

[PATCH 3/6] Don't exit copy-mode when scrolling with mouse-copy-mode set to "within"

2012-09-16 Thread Marcel Partap
--- window-copy.c |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/window-copy.c b/window-copy.c index ca4c97b..bd78f30 100644 --- a/window-copy.c +++ b/window-copy.c @@ -836,7 +836,8 @@ window_copy_mouse( } else if ((m->b & MOUSE_BUTTON) == MOUSE_2) {

[PATCH 2/6] Rename mode-mouse option to mouse-copy-mode

2012-09-16 Thread Marcel Partap
and remove condition to enable mouse in choose mode. This time, no backwards compatible mercy. RTFM! --- input-keys.c|2 +- options-table.c |8 server-client.c |2 +- tmux.1 | 11 +-- window-choose.c |3 +-- window-copy.c |2 +- window.c

Re: using tmux buffers to do a pane search

2012-09-16 Thread Nicholas Marriott
actually try this please which makes paste go through the mode keys path rather than directly to the pane diff --git a/cmd-paste-buffer.c b/cmd-paste-buffer.c index ff92783..2981358 100644 --- a/cmd-paste-buffer.c +++ b/cmd-paste-buffer.c @@ -29,8 +29,8 @@ enum cmd_retval cmd_paste_bu

Re: [PATCH 1/7] Find working path of foreground sub process in osdep_get_cwd

2012-09-16 Thread Nicholas Marriott
I think this makes sense, will look at it more closely tomorrow though. On Sat, Sep 15, 2012 at 03:22:37PM +0200, Marcel Partap wrote: > by picking up the process group leader from the terminal fd > via tcgetpgrp() instead of using the direct pane child process. > > (works on linux, other platfo

Re: [PATCH 3/7] Add mechanism to handle renamed option/value strings in config files.

2012-09-16 Thread Nicholas Marriott
Hi I don't want to carry around backwards compatibility code, at least for very long. So I say either: - No backwards compatibility. - Fix it up in code with strcmp and remove it in no more than two releases. On Sun, Sep 16, 2012 at 06:48:48PM +0200, Marcel Partap wrote: > >> First usage in

Re: [PATCH 1/7] Find working path of foreground sub process in osdep_get_cwd

2012-09-16 Thread Marcel Partap
>> by picking up the process group leader from the terminal fd >> via tcgetpgrp() instead of using the direct pane child process. > > Why? I mean, what problem is this solving? You don't mention it in your > commit message. Nested subshells and su invocations. Consider this: > # root key binding

Re: [PATCH 6/7] Show how VT100 ACS (alternate character set) should actually render

2012-09-16 Thread Marcel Partap
>> +#define CELL_BORDERS " xqlkmjwvtun~" /* " │─┌┐└┘┬┴├┤┼·" */ >> +#define CELL_MARKERS " +, . -" /* " →← ↑ ↓" */ > > And if I don't have a font capable of displaying the characters in the > comments? Then you should upgrade your text editing system of choice to be able to handle 21st cent

Re: [PATCH 3/7] Add mechanism to handle renamed option/value strings in config files.

2012-09-16 Thread Marcel Partap
>> First usage instance is the renaming of the mode-mouse option. > Hmm. I am not sure I'm too keen on this approach. a) Please elaborate. b) Do you have a better idea to warrant config file backwards compability? c) More opinions needed! > What happens when/if > mode-copy-mouse also needs renami