From 4198a19787a8cf54829ac8c2912252f3f27dde2f Mon Sep 17 00:00:00 2001 From: ephphatha Date: Thu, 7 Oct 2021 20:51:53 +1100 Subject: [PATCH] Remove debug assert after picking up an item Doesn't really do anything useful. If the target position doesn't contain an item on the local client the code looks for a matching dropped item and tries to pick that up instead. It doesn't really make sense to assert that an item exists at the target position or that it matches, a desync has already happened by this point if the checks on lines 1647 or 1649 are false. --- Source/inv.cpp | 1 - 1 file changed, 1 deletion(-) diff --git a/Source/inv.cpp b/Source/inv.cpp index 28ccb0b0f..dff9f50d3 100644 --- a/Source/inv.cpp +++ b/Source/inv.cpp @@ -1661,7 +1661,6 @@ void SyncGetItem(Point position, int idx, uint16_t ci, int iseed) return; CleanupItems(&Items[ii], ii); - assert(FindGetItem(idx, ci, iseed) == -1); } bool CanPut(Point position)