Skip to content

[aslspec] fixed tick_loop_limit to match implementation#1770

Merged
Roman-Manevich merged 1 commit intomasterfrom
aslspec-fix
Apr 8, 2026
Merged

[aslspec] fixed tick_loop_limit to match implementation#1770
Roman-Manevich merged 1 commit intomasterfrom
aslspec-fix

Conversation

@Roman-Manevich
Copy link
Copy Markdown
Collaborator

@Roman-Manevich Roman-Manevich commented Mar 31, 2026

The implementation of tick_loop_limit checked whether the decremented counter is greater or equal to 0, whereas the specification wrongfully checked for equality to 0.

@Roman-Manevich Roman-Manevich requested a review from hrutvik March 31, 2026 18:25
@Roman-Manevich Roman-Manevich marked this pull request as ready for review March 31, 2026 18:25
@Roman-Manevich Roman-Manevich marked this pull request as draft March 31, 2026 18:27
@Roman-Manevich Roman-Manevich marked this pull request as ready for review March 31, 2026 18:43
Copy link
Copy Markdown
Collaborator

@HadrienRenaud HadrienRenaud left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks @Roman-Manevich

@Roman-Manevich Roman-Manevich merged commit 921f92b into master Apr 8, 2026
5 checks passed
@Roman-Manevich Roman-Manevich deleted the aslspec-fix branch April 8, 2026 10:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants