On Mon, Dec 29, 2008 at 8:42 PM, Terry Brown <[email protected]>wrote:
>
> run this script once:
>
> if not hasattr(c, 'nn'):
> nn = p.insertAsLastChild()
> nn.setHeadString('here')
> c.nn = nn.copy()
> else:
> assert c.positionExists(c.nn)
> c.selectPosition(c.nn)
The fix is on the trunk at rev 1374. All unit tests pass, including two new
unit tests for c.positionExists. However, this is a major bug fix--as
described below this change has the potential to cause subtle problems.
Please take care, and please report any further problems immediately.
The fix was straightforward, and clearly now does what was originally
intended. However, I was concerned that the existing code, especially the
undo code, might be using c.positionExists in a way that used the old
(buggy) semantics. Happily, a careful review of the code indicates that
this apparently does not happen.
The fix has a good chance of fixing one or more problems with the qt gui
code, which uses c.positionExists as a safeguard. Clearly, that safeguard
was failing before.
Edward
--~--~---------~--~----~------------~-------~--~----~
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
-~----------~----~----~----~------~----~------~--~---
