should it show an error? On Sun, Feb 16, 2014 at 11:12:06PM +0000, Thomas Adam wrote: > On Sun, Feb 16, 2014 at 07:39:27AM -0800, Felix Rosencrantz wrote: > > I saw it when setting a colour option to a bogus value "none". > > So why not send a patch through? > > Nicholas, see below. I can't say I like it, but it does smooth over the > cracks. > > -- Thomas Adam > > -- > "Deep in my heart I wish I was wrong. But deep in my heart I know I am > not." -- Morrissey ("Girl Least Likely To" -- off of Viva Hate.)
> diff --git a/cmd-set-option.c b/cmd-set-option.c > index 15b14f0..5ae4e5c 100644 > --- a/cmd-set-option.c > +++ b/cmd-set-option.c > @@ -307,11 +307,13 @@ cmd_set_option_set(struct cmd *self, struct cmd_q *cmdq, > break; > case OPTIONS_TABLE_COLOUR: > o = cmd_set_option_colour(self, cmdq, oe, oo, value); > - style_update_new(oo, o->name, oe->style); > + if (o != NULL) > + style_update_new(oo, o->name, oe->style); > break; > case OPTIONS_TABLE_ATTRIBUTES: > o = cmd_set_option_attributes(self, cmdq, oe, oo, value); > - style_update_new(oo, o->name, oe->style); > + if (o != NULL) > + style_update_new(oo, o->name, oe->style); > break; > case OPTIONS_TABLE_FLAG: > o = cmd_set_option_flag(self, cmdq, oe, oo, value); ------------------------------------------------------------------------------ Android apps run on BlackBerry 10 Introducing the new BlackBerry 10.2.1 Runtime for Android apps. Now with support for Jelly Bean, Bluetooth, Mapview and more. Get your Android app in front of a whole new audience. Start now. http://pubads.g.doubleclick.net/gampad/clk?id=124407151&iu=/4140/ostg.clktrk _______________________________________________ tmux-users mailing list tmux-users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/tmux-users