Understand, thanks for the explanation.

On Wed, Mar 06, 2013 at 11:26:30AM +0000, Nicholas Marriott wrote:
> On Wed, Mar 06, 2013 at 08:02:49PM +0900, eugene everson wrote:
> > Hi Nicholas,
> > 
> > First just quick want to say thanks first, switched yesterday to tmux and
> > it's been so sweet. And also love the source code. Thanks.
> > 
> > About the patch, this is a bit nitpicking so if you're busy think you can
> > leave this safely for later.
> > 
> > - Later in grid_string_cells if(with_codes) is true it dereference lastgc
> > again twice. At the moment it's not called anywhere with if(with_codes) ==
> > true and lastgc == NULL but don't know if that's possible in the future.
> 
> It won't be possible and anyone who tries it will find out that it
> crashes :-). The last cell is only needed when generating escape
> sequences (with_codes = 1), so that the function knows what the state is
> across multiple lines and doesn't have to generate extra escape
> sequences.
> 
> > 
> > - In cmd_capture_pane_exec it calls grid_string_cells the same way as
> > the patch I sent. Guess that could be brought in line with just passing
> > NULL with your change.
> 
> capture-pane does set with_codes to 1 (if you give -e flag) so it needs
> to pass in a lastgc pointer.
> 
> Thanks
> 
> > 
> > Eugene.
> > 
> > On Wed, Mar 06, 2013 at 09:57:13AM +0000, Nicholas Marriott wrote:
> > > Thanks - fixed by checking in grid_string_cells instead.
> > > 
> > > 
> > > 
> > > On Wed, Mar 06, 2013 at 10:24:55AM +0900, eugene everson wrote:
> > > > Reproducible with :find-window .. (<ctrl-b>f)
> > > > ---
> > > >  grid-view.c |    4 +++-
> > > >  1 file changed, 3 insertions(+), 1 deletion(-)
> > > > 
> > > > diff --git a/grid-view.c b/grid-view.c
> > > > index 7ef443a..c639b2a 100644
> > > > --- a/grid-view.c
> > > > +++ b/grid-view.c
> > > > @@ -229,10 +229,12 @@ grid_view_delete_cells(struct grid *gd, u_int px, 
> > > > u_int py, u_int nx)
> > > >  char *
> > > >  grid_view_string_cells(struct grid *gd, u_int px, u_int py, u_int nx)
> > > >  {
> > > > +       struct grid_cell *gc = NULL;
> > > > +
> > > >         GRID_DEBUG(gd, "px=%u, py=%u, nx=%u", px, py, nx);
> > > >  
> > > >         px = grid_view_x(gd, px);
> > > >         py = grid_view_y(gd, py);
> > > >  
> > > > -       return (grid_string_cells(gd, px, py, nx, NULL, 0, 0, 0));
> > > > +       return (grid_string_cells(gd, px, py, nx, &gc, 0, 0, 0));
> > > >  }
> > > > -- 
> > > > 1.7.9.5
> > > > 
> > > > 
> > > > ------------------------------------------------------------------------------
> > > > Symantec Endpoint Protection 12 positioned as A LEADER in The Forrester 
> > > >  
> > > > Wave(TM): Endpoint Security, Q1 2013 and "remains a good choice" in the 
> > > >  
> > > > endpoint security space. For insight on selecting the right partner to 
> > > > tackle endpoint security challenges, access the full report. 
> > > > http://p.sf.net/sfu/symantec-dev2dev
> > > > _______________________________________________
> > > > tmux-users mailing list
> > > > tmux-users@lists.sourceforge.net
> > > > https://lists.sourceforge.net/lists/listinfo/tmux-users

------------------------------------------------------------------------------
Symantec Endpoint Protection 12 positioned as A LEADER in The Forrester  
Wave(TM): Endpoint Security, Q1 2013 and "remains a good choice" in the  
endpoint security space. For insight on selecting the right partner to 
tackle endpoint security challenges, access the full report. 
http://p.sf.net/sfu/symantec-dev2dev
_______________________________________________
tmux-users mailing list
tmux-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/tmux-users

Reply via email to