Skip to content

Commit

Permalink
Remove a duplicate StorageGroup sync worker, merge both storage regions
Browse files Browse the repository at this point in the history
It seems a second StorageGroup sync worker was added recently, but the old one was kept in. I've removed the old one since the new one seems to be better (as it doesn't rely on a `Thing`, just a map and ID).

On top of that, I've merged the two regions named Storage.
  • Loading branch information
SokyranTheDragon committed Apr 28, 2024
1 parent 67d1537 commit bfd8df1
Showing 1 changed file with 0 additions and 6 deletions.
6 changes: 0 additions & 6 deletions Source/Client/Syncing/Dict/SyncDictRimWorld.cs
Original file line number Diff line number Diff line change
Expand Up @@ -1091,9 +1091,7 @@ public static class SyncDictRimWorld
return data.MpContext().map.storageGroups.groups.Find(g => g.loadID == loadId);
}
},
#endregion

#region Storage
{
(ByteWriter data, StorageSettings storage) => {
WriteSync(data, storage.owner);
Expand All @@ -1103,10 +1101,6 @@ public static class SyncDictRimWorld
return parent?.GetStoreSettings();
}
},
{
(ByteWriter data, StorageGroup group) => WriteSync(data, group.members.First()),
(ByteReader data) => ReadSync<IStorageGroupMember>(data).Group
},
#endregion

#region Letters
Expand Down

0 comments on commit bfd8df1

Please sign in to comment.