Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Breakpoint position from cursor #10812

Merged

Conversation

carolahp
Copy link
Contributor

Improving algoriths to calculate best node from caret position,
Restructuring debugging menu

carolahp and others added 6 commits January 24, 2022 10:42
…the caret.

In this solution, sequence nodes search first among their statements.
Including tests for this new behavior
Updating debugging submenu labels and icons
…enu in two: Breakpoints and Advanced Debugging. Placing 'counter' command in the root menu.
@MarcusDenker
Copy link
Member

MarcusDenker commented Jan 27, 2022

Test failing:

  • testBestNodeFor – RASTTest
  • testBestNodeFor - RBParserTest
  • testMoveDefinition – RBMoveTemporaryVariableDefinitionTransformationTest
  • testMoveDefinitionIntoBlockThatIsAReceiverOfACascadedMessage - RBMoveTemporaryVariableDefinitionTransformationTest

@MarcusDenker MarcusDenker added the Status: Need more work The issue is nearly ready. Waiting some last bits. label Jan 27, 2022
Creating "Reflective" submenu containing Metalink related commands.
This submenu is not shown by default. Creating a Pharo setting to show/hide it.
Placing Counter and Watchpoint commands in "Debugging" submenu.
@MarcusDenker
Copy link
Member

Just one failing test left:

  • RBMoveTemporaryVariableDefinitionTransformationTest>>testMoveDefinition

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Need more work The issue is nearly ready. Waiting some last bits.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants