On Mon, Jan 12, 2009 at 6:04 PM, Terry Brown <[email protected]>wrote:

>
> Ok - but assuming the code works for top level positions (it should,
> right?) it still doesn't make sense to pop from an empty list and raise
> an exception.


Oops.  You are right.  I just tested this and got a complaint that v.parents
doesn't exist. So obviously the code is completely untested.

I see where the confusion lies.  There are unit tests for item2position and
position2item in the Qt gui, but of course they are irrelevant.  I was
confusing these tests with the test for c.vnode2position.  But there is no
such test.

Sorry about that.  Let me fix this on the trunk.  I should be able to do so
today.

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
-~----------~----~----~----~------~----~------~--~---

Reply via email to