Network Finance saving#418
Conversation
| return should_save | ||
|
|
||
| //**Used to check if a movable will be saved by world save. */ | ||
| /atom/movable/proc/will_save() |
There was a problem hiding this comment.
What is that proc for? It's a bit awkwardly named, and its description make it sound like it's identical to should_save() ?
Like what's the difference between that, and should_save()?
There was a problem hiding this comment.
This proc determines whether or not an atom will actually be saved during world save iteration, meaning it's either in a saved z-level or saved area. For example it is now used to determine if a mob will be saved by the world save or if it needs to be saved in limbo, and also used to determine if extensions should self-report for saving depending on whether or not their holder will save.
There was a problem hiding this comment.
Alright, but can you put that in the comments and give it a name that's a bit more representative of that? Like maybe "should_save_in_limbo", or something else?
PsyCommando
added
the
💲 economy
2 participants
Description of changes
Adds saved variables for network finance, with some adjustments. Fixes a few issues with flattened object and extension saving. Some diff noise in the _persistence.dme file due to automatic alphabetical sorting.
Authorship
Myself