Does this work? If we do this can probably drop lastattr too and maybe
move the memcpy inside the _code func.
diff --git a/grid.c b/grid.c
index a84a58b..12ec643 100644
--- a/grid.c
+++ b/grid.c
@@ -477,7 +477,7 @@ grid_string_cells_bg(const struct grid_cell *gc, int
*values)
* bytes.
*/
void
-grid_string_cells_code(const struct grid_cell *lastgc,
+grid_string_cells_code(struct grid_cell *lastgc,
const struct grid_cell *gc, char *buf, size_t len, int escape_c0)
{
int oldc[16], newc[16], s[32];
@@ -505,6 +505,8 @@ grid_string_cells_code(const struct grid_cell *lastgc,
if (!(attr & attrs[i].mask) && (lastattr & attrs[i].mask)) {
s[n++] = 0;
lastattr &= GRID_ATTR_CHARSET;
+ lastgc->fg = lastgc->bg = 7;
+ lastgc->flags &= ~(GRID_FLAG_FG256|GRID_FLAG_BG256);
break;
}
}
On Wed, Sep 17, 2014 at 06:31:20PM +0000, Matthew Darby wrote:
> That's correct. In fact, after a test, it adds \e[0;39;49m if the previous
> cell had no colour. There's two issues I can see:
>
> 1) If the previous cell's colour was default, it will add the above default
> colours superfluously.
> 2) If the current cell colour changes in addition to the attribute being
> removed, you get a code like \0;39;49;31m. This is a superfluous colour
> change.
>
> I'm not sure how to proceed without restructuring the function (it's well
> suited to the structure it already has). I'm very open to suggestions!
>
> ---- On Wed, 17 Sep 2014 15:30:04 +0000 Nicholas Marriott
> <nicholas.marri...@gmail.com> wrote ----
>
>
> > If the cell is default fg/bg, won't this add \e[0m\e[39m\[49m? The two
> > colour changes are unnecessary.
> >
> >
> > On Tue, Sep 16, 2014 at 06:39:38PM +0000, Matthew Darby wrote:
> > > Hi,
> > > grid_string_cells_code in grid.c currently resets all attributes when
> it
> > > finds the current cell lacks an attribute from the previous cell.
> > > Remaining non-colour attributes seem to be well-looked after, but any
> > > colours set are lost. The supplied patch restores the colours from
> the
> > > previous cell into the current cell. This seems to be fine even if
> the
> > > current cell also changes the colours, because the previously set
> colours
> > > will be ignored.
> > > Thanks,
> > > Matthew Darby
> >
> > > _______________________________________________
> > > tmux-users mailing list
> > > tmux-users@lists.sourceforge.net
> > > https://lists.sourceforge.net/lists/listinfo/tmux-users
> >
> >
>
------------------------------------------------------------------------------
Want excitement?
Manually upgrade your production database.
When you want reliability, choose Perforce
Perforce version control. Predictably reliable.
http://pubads.g.doubleclick.net/gampad/clk?id=157508191&iu=/4140/ostg.clktrk
_______________________________________________
tmux-users mailing list
tmux-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/tmux-users
