April White:

> Off the cuff, I'd say that UI events should not be able to add multiple
> commands, but of course that is easier said than done.  If the same
> "executing" value is used, then tool commmand could still be blocked,
> and only an extender or internal events such as "command.go.needs" would
> be able to add multiple commands.

   The implementation of the 'executing' state will become more complex.

> I think you mis-read my message.  Currently most/all calls of
> AddCommand() are followed by "if (commandCurrent > 0)" before calling
> Execute.  If the UI thread monitors the queue and calls Execute when
> necessary, then the call to Execute after every AddCommand would be
> unnecessary.

   There is also the handling of the isBuilding and isBuilt flags
although that doesn't appear very robust to me as they are not
reinitialised when switching buffers.

   Neil

_______________________________________________
Scite-interest mailing list
Scite-interest@lyra.org
http://mailman.lyra.org/mailman/listinfo/scite-interest

Reply via email to