Re: [patch] more dispatch stuff

[Angus Leeming]

Lars Gullik Bjønnes wrote:

> 
> This replaces dispatch_result enum with a real type, forward
> declared when possible.

Also, you don't need to use DispatchResult in the body of the code, 
just when passing it around. The operator dispatch_result_t() means 
that things will just work.

For example, I would get rid of the second change below. It isn't 
needed.

-dispatch_result
+DispatchResult
 InsetCollapsable::priv_dispatch(FuncRequest const & cmd,
                                 idx_type & idx, pos_type & pos)
 {
@@ -355,7 +356,7 @@ InsetCollapsable::priv_dispatch(FuncRequ
                         return DISPATCHED;
 
                 default:
-                        dispatch_result result = inset.dispatch(cmd);
+                        DispatchResult result = inset.dispatch(cmd);
                         if (result >= FINISHED)
                                 bv->unlockInset(this);
-- 
Angus