Add new AST node for continue statement in while loops #4
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR adds comprehensive support for the continue statement to the O²L programming language, enabling users to skip the current iteration of a while loop and proceed to the next iteration. This enhances control flow capabilities by providing a way to filter data and handle conditional processing within loops.
Key changes include:
- Implementation of
ContinueNodeAST class with exception-based control flow - Lexer and parser updates to recognize and parse the
continuekeyword - Integration with the existing while loop implementation to catch and handle continue exceptions
Reviewed Changes
Copilot reviewed 13 out of 13 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| src/AST/ContinueNode.hpp | Defines the AST node class for continue statements |
| src/AST/ContinueNode.cpp | Implements continue statement evaluation by throwing ContinueException |
| src/Common/Exceptions.hpp | Adds ContinueException for control flow handling |
| src/AST/WhileStatementNode.cpp | Updates while loop to catch and handle continue exceptions |
| src/Lexer.hpp | Adds CONTINUE token type to the lexer |
| src/Lexer.cpp | Maps "continue" keyword to CONTINUE token type |
| src/Parser.hpp | Declares parseContinueStatement method |
| src/Parser.cpp | Implements continue statement parsing and AST node creation |
| tests/test_continue_statement.cpp | Comprehensive test suite for continue statement functionality |
| tests/CMakeLists.txt | Adds continue statement tests to build configuration |
| examples/test_continue_examples.obq | Example file demonstrating various continue statement use cases |
| README.md | Updates documentation with continue statement features and examples |
| CMakeLists.txt | Includes new ContinueNode source and header files in build |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Co-authored-by: Copilot <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request adds support for the
continuestatement to the language, enabling users to skip to the next iteration of awhileloop for more flexible control flow. The implementation covers lexer, parser, AST, runtime, documentation, and testing, ensuring thatcontinueworks seamlessly alongside existing features likebreak. The documentation and examples are updated to help users understand and use the new statement effectively.Language Feature Implementation
ContinueNodeto the AST (src/AST/ContinueNode.cpp,src/AST/ContinueNode.hpp) and corresponding runtime logic, throwing a newContinueExceptionto handle control flow in loops. [1] [2] [3]continuekeyword, parse it into AST nodes, and handle it during statement parsing. [1] [2] [3] [4] [5] [6]Control Flow Integration
WhileStatementNodeto catchContinueExceptionand skip to the next iteration, ensuring correct loop semantics.Documentation and Examples
README.mdto document the newcontinuestatement, its features, usage patterns, and added multiple example scenarios. [1] [2] [3] [4]examples/test_continue_examples.obqdemonstrating practical uses ofcontinuein filtering, validation, searching, batching, and combined withbreak.Build and Testing
CMakeLists.txt,tests/CMakeLists.txt) to include new source and header files, and added a dedicated test for continue statement functionality. [1] [2] [3] [4]