On Wed, Jul 02, 2003 at 03:04:49PM +0200, Andre Poenitz spake thusly:
>
> On Wed, Jul 02, 2003 at 04:24:26PM +0300, Martin Vermeer wrote:
> > Don't you think I've tried?
>
> Not too hard, actually.
>
> > Currently I have
> >
> > switch (cmd.action) {
> > case LFUN_INSET_MODIFY:
> > {
> > InsetNoteParams params;
> > InsetNoteMailer::string2params(cmd.argument, params);
> > params_.type = params.type;
> > params_.branch = params.branch;
> > setButtonLabel();
> > bv->updateInset(this);
> > return DISPATCHED;
> > }
> > case LFUN_INSET_EDIT:
> > if (cmd.button() == mouse_button::button3)
> > break;
>
> So guess what ^^^^^^ this looks like.
>
> [Hint: That's not a "return"]
>
> Andre'Oops... yes. It's the surrounding routine demanding a value, not the switch as such that is wrong! Is it OK to substitute "return UNDISPATCHED" here? It compiles, but... Martin
pgp00000.pgp
Description: PGP signature
