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

Timed infinite loop fix #80

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Timed infinite loop fix #80

wants to merge 4 commits into from

Commits on Jan 12, 2024

  1. Configuration menu
    Copy the full SHA
    60c123d View commit details
    Browse the repository at this point in the history
  2. Add test for issue #79

    Not currently failing, just demonstrates the unfortunate behavior.
    strohel committed Jan 12, 2024
    Configuration menu
    Copy the full SHA
    e6fb7df View commit details
    Browse the repository at this point in the history
  3. timed: fix #79 by enqueueing (rather than handling) delayed messages …

    …when due
    
    Also rename `fire_at` to `enqueue_at` to be explicit about the fact.
    
    This is a trade-off that prevents 2 sorts of bad behavior:
    - actors with send-sending messages never handling any delayed/recurring messages (#72)
    - actors with recurring messages that are slower to handle than their interval eventually not processing any outside messages (#79)
    
    See the tweaked tests for the change of behavior.
    strohel committed Jan 12, 2024
    Configuration menu
    Copy the full SHA
    66fcd74 View commit details
    Browse the repository at this point in the history

Commits on Jan 16, 2024

  1. Configuration menu
    Copy the full SHA
    b4b0bc2 View commit details
    Browse the repository at this point in the history