Yes you're right this is fine thanks.
On Mon, Feb 17, 2014 at 03:05:21PM +0000, Thomas Adam wrote:
> Hi,
>
> It does show an error when the command is parsed. Or were you wanting
> something explicit here?
>
> Thomas Adam
>
> On 17 Feb 2014 13:49, "Nicholas Marriott" <[1]nicholas.marri...@gmail.com>
> wrote:
> >
> > 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.
> >
>
> [2]http://pubads.g.doubleclick.net/gampad/clk?id=124407151&iu=/4140/ostg.clktrk
> > _______________________________________________
> > tmux-users mailing list
> > [3]tmux-users@lists.sourceforge.net
> > [4]https://lists.sourceforge.net/lists/listinfo/tmux-users
>
> References
>
> Visible links
> 1. mailto:nicholas.marri...@gmail.com
> 2.
> http://pubads.g.doubleclick.net/gampad/clk?id=124407151&iu=/4140/ostg.clktrk
> 3. mailto:tmux-users@lists.sourceforge.net
> 4. https://lists.sourceforge.net/lists/listinfo/tmux-users
------------------------------------------------------------------------------
Managing the Performance of Cloud-Based Applications
Take advantage of what the Cloud has to offer - Avoid Common Pitfalls.
Read the Whitepaper.
http://pubads.g.doubleclick.net/gampad/clk?id=121054471&iu=/4140/ostg.clktrk
_______________________________________________
tmux-users mailing list
tmux-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/tmux-users
