#5246 basic-46.1 hangs

current: 8.6.0
closed-fixed
Don Porter
8
2013-06-05
2013-06-05
Don Porter
No

On the trunk, in a --enable-symbols=all build,
test basic-46.1 never returns.

Discussion

  • Don Porter
    Don Porter
    2013-06-05

    [gets $f] is never reading "DONE"

     
  • Don Porter
    Don Porter
    2013-06-05

    And the root cause is this:

    $ make shell
    ...
    % continue
    bad stack adjustment when compiling continue (was 2 instead of 1)

     
  • Don Porter
    Don Porter
    2013-06-05

    Just a stray TclAdjustStackDepth left over...

     
  • Yeah, I've screwed up. Backed it out of trunk; will fix.

     
  • Don Porter
    Don Porter
    2013-06-05

    Look at what fossil just did!

     
  • Don Porter
    Don Porter
    2013-06-05

    The logic of the checkStack argument to
    ValidatePcAndStackTop is out of sync. A check
    is made right after INST_EXPAND_DROP
    that causes trouble.

     
  • Don Porter
    Don Porter
    2013-06-05

    see branch bug-3614342

     
  • Don Porter
    Don Porter
    2013-06-05

    fix merged to trunk.

     
  • Don Porter
    Don Porter
    2013-06-05

    • assigned_to: dkf --> dgp
    • status: open --> closed-fixed