The tty code already stores attributes and doesn't modify them if unaltered, so the screen drawing code doesn't need to do so too. This probably also fixes bugs when switching screens where the actual tty attributes may not have been what the draw code (erroneously) expected.

This commit is contained in:
Nicholas Marriott 2007-12-02 18:23:10 +00:00
parent ba56989d0f
commit 8419e99789
2 changed files with 5 additions and 15 deletions

View File

@ -1,4 +1,4 @@
/* $Id: screen.c,v 1.54 2007-11-30 11:08:35 nicm Exp $ */
/* $Id: screen.c,v 1.55 2007-12-02 18:23:10 nicm Exp $ */
/*
* Copyright (c) 2007 Nicholas Marriott <nicm@users.sourceforge.net>
@ -317,9 +317,7 @@ screen_draw_start(struct screen_draw_ctx *ctx, struct screen *s,
ctx->sel.flag = 0;
ctx->attr = s->attr;
ctx->colr = s->colr;
ctx->write(ctx->data, TTY_ATTRIBUTES, s->attr, s->colr);
ctx->write(ctx->data, TTY_SCROLLREGION, 0, screen_last_y(s));
ctx->write(ctx->data, TTY_CURSOROFF);
ctx->write(ctx->data, TTY_MOUSEOFF);
@ -404,7 +402,6 @@ screen_draw_stop(struct screen_draw_ctx *ctx)
if (ctx->cx != s->cx || ctx->cy != s->cy)
ctx->write(ctx->data, TTY_CURSORMOVE, s->cy, s->cx);
if (ctx->attr != s->attr || ctx->colr != s->colr)
ctx->write(ctx->data, TTY_ATTRIBUTES, s->attr, s->colr);
if (s->mode & MODE_BACKGROUND) {
@ -517,11 +514,7 @@ void
screen_draw_set_attributes(
struct screen_draw_ctx *ctx, u_char attr, u_char colr)
{
if (attr != ctx->attr || colr != ctx->colr) {
ctx->write(ctx->data, TTY_ATTRIBUTES, attr, colr);
ctx->attr = attr;
ctx->colr = colr;
}
}
/* Draw single cell. */

5
tmux.h
View File

@ -1,4 +1,4 @@
/* $Id: tmux.h,v 1.107 2007-11-30 11:08:35 nicm Exp $ */
/* $Id: tmux.h,v 1.108 2007-12-02 18:23:10 nicm Exp $ */
/*
* Copyright (c) 2007 Nicholas Marriott <nicm@users.sourceforge.net>
@ -406,9 +406,6 @@ struct screen_draw_ctx {
u_int ox;
u_int oy;
u_char attr;
u_char colr;
struct screen_draw_sel sel;
};