On Thursday, April 6, 2017 at 4:34:07 AM UTC-5, Edward K. Ream wrote:

> A big oops:  There are two more *important *uses of atAuto in 
ic.createOutline: [snip]

A giant step forward.  All unit tests pass when ic.createOutline asserts 
atAuto is False.

To make this happen, ic.scannerUnitTest calls 
ic.createOutline(...atAuto=True), *regardless *of its atAuto keyword arg.

This means:

1. All atAuto args to ic.scannerUnitTest can be eliminated.

This is a big deal: all mysterious atAuto args in the unit tests *themselves 
*will disappear.

2. All atAuto args to all importers can be eliminated.

Another big deal. This was a major "huh?".

3. The following code in i.gen_ref never fires and can be eliminated:

    if self.is_rst and not self.atAuto:
        return None

Not a huge deal, but it's always good to remove a mysterious test and 
special case.

We are about to see the end of a major "huh?" that has infected Leo's code.

Edward

-- 
You received this message because you are subscribed to the Google Groups 
"leo-editor" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at https://groups.google.com/group/leo-editor.
For more options, visit https://groups.google.com/d/optout.

Reply via email to