From 9ec457575d0db6450d8a436426b4a705c49285aa Mon Sep 17 00:00:00 2001 From: Tiago Cunha Date: Fri, 25 Nov 2011 13:30:45 +0000 Subject: [PATCH] Sync OpenBSD patchset 979: Make window_pane_index work the same as window_index, from Ben Boeckel. --- cmd-respawn-pane.c | 7 +++++-- cmd-split-window.c | 3 ++- screen-redraw.c | 5 +++-- status.c | 5 ++++- tmux.h | 2 +- window.c | 18 ++++++++++-------- 6 files changed, 25 insertions(+), 15 deletions(-) diff --git a/cmd-respawn-pane.c b/cmd-respawn-pane.c index 7e272a80..d4d67d54 100644 --- a/cmd-respawn-pane.c +++ b/cmd-respawn-pane.c @@ -49,15 +49,18 @@ cmd_respawn_pane_exec(struct cmd *self, struct cmd_ctx *ctx) struct session *s; struct environ env; const char *cmd; - char *cause; + char *cause; + u_int idx; if ((wl = cmd_find_pane(ctx, args_get(args, 't'), &s, &wp)) == NULL) return (-1); w = wl->window; if (!args_has(self->args, 'k') && wp->fd != -1) { + if (window_pane_index(wp, &idx) != 0) + fatalx("index not found"); ctx->error(ctx, "pane still active: %s:%u.%u", - s->name, wl->idx, window_pane_index(w, wp)); + s->name, wl->idx, idx); return (-1); } diff --git a/cmd-split-window.c b/cmd-split-window.c index 6a95e1ba..258c6327 100644 --- a/cmd-split-window.c +++ b/cmd-split-window.c @@ -139,7 +139,8 @@ cmd_split_window_exec(struct cmd *self, struct cmd_ctx *ctx) environ_free(&env); if (args_has(args, 'P')) { - paneidx = window_pane_index(wl->window, new_wp); + if (window_pane_index(new_wp, &paneidx) != 0) + fatalx("index not found"); ctx->print(ctx, "%s:%u.%u", s->name, wl->idx, paneidx); } return (0); diff --git a/screen-redraw.c b/screen-redraw.c index a12fa365..c9e05420 100644 --- a/screen-redraw.c +++ b/screen-redraw.c @@ -264,7 +264,7 @@ screen_redraw_draw_number(struct client *c, struct window_pane *wp) { struct tty *tty = &c->tty; struct session *s = c->session; - struct options *oo = &s->options; + struct options *oo = &s->options; struct window *w = wp->window; struct grid_cell gc; u_int idx, px, py, i, j, xoff, yoff; @@ -272,7 +272,8 @@ screen_redraw_draw_number(struct client *c, struct window_pane *wp) char buf[16], *ptr; size_t len; - idx = window_pane_index(w, wp); + if (window_pane_index(wp, &idx) != 0) + fatalx("index not found"); len = xsnprintf(buf, sizeof buf, "%u", idx); if (wp->sx < len) diff --git a/status.c b/status.c index 5db40b4b..ca3e9d43 100644 --- a/status.c +++ b/status.c @@ -372,6 +372,7 @@ status_replace1(struct client *c, struct session *s, struct winlink *wl, char ch, tmp[256], *ptr, *endptr, *freeptr; size_t ptrlen; long limit; + u_int idx; if (s == NULL) s = c->session; @@ -422,8 +423,10 @@ status_replace1(struct client *c, struct session *s, struct winlink *wl, ptr = tmp; goto do_replace; case 'P': + if (window_pane_index(wp, &idx) != 0) + fatalx("index not found"); xsnprintf( - tmp, sizeof tmp, "%u", window_pane_index(wl->window, wp)); + tmp, sizeof tmp, "%u", idx); ptr = tmp; goto do_replace; case 'S': diff --git a/tmux.h b/tmux.h index 08650d51..be8f296c 100644 --- a/tmux.h +++ b/tmux.h @@ -1927,7 +1927,7 @@ struct window_pane *window_pane_next_by_number(struct window *, struct window_pane *, u_int); struct window_pane *window_pane_previous_by_number(struct window *, struct window_pane *, u_int); -u_int window_pane_index(struct window *, struct window_pane *); +int window_pane_index(struct window_pane *, u_int *); u_int window_count_panes(struct window *); void window_destroy_panes(struct window *); struct window_pane *window_pane_find_by_id(u_int); diff --git a/window.c b/window.c index 8b82c06e..e170dd27 100644 --- a/window.c +++ b/window.c @@ -483,19 +483,21 @@ window_pane_previous_by_number(struct window *w, struct window_pane *wp, return (wp); } -u_int -window_pane_index(struct window *w, struct window_pane *wp) +int +window_pane_index(struct window_pane *wp, u_int *i) { struct window_pane *wq; - u_int n; + struct window *w = wp->window; - n = options_get_number(&w->options, "pane-base-index"); + *i = options_get_number(&w->options, "pane-base-index"); TAILQ_FOREACH(wq, &w->panes, entry) { - if (wp == wq) - break; - n++; + if (wp == wq) { + return (0); + } + (*i)++; } - return (n); + + return (-1); } u_int