Skip to content

[#293] Treat DIP body as new stack frame. Fixed some related issues

Ilya Peresadin requested to merge pva/#293-nested-frames into master

Description

Problem: we don't have stack frames for now.

Solution: let's treat DIP body as new frame and show it on the plugin.

Also, related bug about @WithLoc@ discovered and fixed: it's treaded not uniformly with other meta instructions, and it caused invalid behaviour of the debugger in some cases.

Related issue(s)

Resolves #293 (closed)

Checklist for your Merge Request

Related changes (conditional)

  • Tests (see short guidelines)

    • If I added new functionality, I added tests covering it.
    • If I fixed a bug, I added a regression test to prevent the bug from silently reappearing again.
  • Documentation

    • I checked whether I should update the docs and did so if necessary:
    • I updated changelog files of all affected packages released to Hackage if my changes are externally visible.

Stylistic guide (mandatory)

Edited by Ilya Peresadin

Merge request reports