Hi Unfortunately I don't have the backtrace anymore, I was busy and the easiest thing at the time was to revert it, it was definitely under grid_view_insert_cells and I couldn't see anything obvious.
I only saw it very occasionally and never managed to reproduce - I know it was crashing regularly to someone else in zenicb in emacs on OpenBSD along with occasional screen corruption - I suspect it is some edge case that emacs sends under rare circumstances. tools/fuzz.c (possibly modified to favour insert/delete cells?) might turn something up but if you don't mind about the feature, don't worry about it - we can just leave it reverted. We'd need to be very sure it was right before it went in again anyway. On Sat, Jan 24, 2015 at 11:53:20PM +0000, Balazs Kezes wrote: > On 2015-01-06 21:31 +0000, Nicholas Marriott wrote: > > This was causing random crashes, so it's clearly still wrong and I've > > reverted back to before any of these changes. I don't have time to > > look into it further but maybe you want to have a look (with valgrind > > maybe?). > > I had no luck reproducing. Neither did valgrind show any suspicious > behavior. I presume you don't have a stack trace, right? > > Anyways, I'm fine having this feature reverted as it is harmless and > wasn't that important for me as my other patch from this series. :) > > Thanks! > > -- > Balazs ------------------------------------------------------------------------------ New Year. New Location. New Benefits. New Data Center in Ashburn, VA. GigeNET is offering a free month of service with a new server in Ashburn. Choose from 2 high performing configs, both with 100TB of bandwidth. Higher redundancy.Lower latency.Increased capacity.Completely compliant. http://p.sf.net/sfu/gigenet _______________________________________________ tmux-users mailing list tmux-users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/tmux-users
