On Jan 29, 12:42 pm, "Edward K. Ream" <[email protected]> wrote:
> My plan is to un-revert this morning's work, put a 'return' at the > start of onItemChanged, and rerun all the unit tests. Wish me luck. That worked. Hooray! Rev 1456 contains the recent work, but the 'if self.busy():' pattern has been *disabled*. All unit tests pass. Please report any problem immediately. I don't expect any. I plan now to *carefully* reintroduce the 'if self.busy()' pattern. Edward P.S. Disabling the onItemChanged event handler is a happy consequence of solving the headline-reversion bug. Indeed, it doesn't matter now when onHeadChanged is called, provided it *is* called sometime while switching nodes. This is a fairly important collapse of complexity, and we see now that it allows us to removed onItemChanged without any qualms. And removing onItemChanged is a big deal: we don't have to worry about locking it out. So this is already better than I hoped. EKR --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "leo-editor" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/leo-editor?hl=en -~----------~----~----~----~------~----~------~--~---
