Skip to content
Snippets Groups Projects

client: Clean up implementation of Pause action and fix double wait

Merged Clément Pit-Claudel requested to merge cpc/pause into main
All threads resolved!
  • js/src/main/scala/cs214/webapp/client/StateMachineClientApp.scala:
    (processing): New Boolean indicating that an event loop is running.
    (timeout, setCooldown, tryProcessNextAction): Remove.
    (processActions): New function: start an event loop, setting processing to true while traversing actionsQueue. Recurse through a future to allow individual actions to delay the next iteration of the event loop.
    (processAction): Return a Future to let individual actions handle timeouts.
    (after): New function: Expose setTimeout as a Future.

Reported-by: Simon Lefort androz2091@gmail.com

Edited by Clément Pit-Claudel

Merge request reports

Approved by

Merged by Clément Pit-ClaudelClément Pit-Claudel 6 months ago (Dec 4, 2024 11:55pm UTC)

Merge details

  • Changes merged into main with 3e0dd930.
  • Deleted the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Clément Pit-Claudel resolved all threads

    resolved all threads

  • merged manually

  • Please register or sign in to reply
    Loading