implement-state-merge #27
No reviewers
Labels
No labels
bug
duplicate
enhancement
help wanted
invalid
priority
high
priority
low
priority
medium
question
wontfix
No milestone
No project
No assignees
2 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
TTK4145/elevator!27
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "implement-state-merge"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Blocked by #25
Closes #9
Related to #4
WIP: implement-state-mergeto implement-state-merge@ -486,2 +689,3 @@// Serialize the data// Serialize each section of the state to get a size breakdownprintln!(thought lights were removed? to send
me problem apparently
@ -259,0 +298,4 @@// Elevator must not be obstructedmatch self.state {// TODO: Testtodo!
@ -259,0 +305,4 @@_ => {}}// TODO: Testtototodo!
@ -353,0 +480,4 @@}/// Update the last seen value for an Elevatorpub fn see_call(&mut self, seen_by: ElevatorId, state: CallState) {not sure i like the name see call.
not descriptive enough
Skill issue in English, marking as resolved.
@ -353,0 +536,4 @@/// NOTE: I know this is a horrible function name, becuase it does check a lot of things./// Honestly, I just couldn't be bothered to think of a better name...////// TODO: Find better namenon blocking: agree, but no idea what
update unchecked values?
We'll leave the TODO and fix in cleanup.