Re: [PATCH 1/1] choose-list: Don't segfault on empty list

2012-09-05 Thread Thomas Adam
Hi, On 5 September 2012 11:04, Nicholas Marriott wrote: > This looks wrong - the first item now has index (u_int)-1? Yeah -- wasn't thinking properly. -- Thomas Adam -- Live Security Virtual Conference Exclusive live e

Re: [PATCH 1/1] choose-list: Don't segfault on empty list

2012-09-05 Thread Nicholas Marriott
This looks wrong - the first item now has index (u_int)-1? idx will still be 0 if there was nothing added, so why not just this? Index: cmd-choose-list.c === RCS file: /cvs/src/usr.bin/tmux/cmd-choose-list.c,v retrieving revision 1.2