Feature/snooping use range#1183
Feature/snooping use range#1183raydienull wants to merge 6 commits intoSphereserver:masterfrom raydienull:feature/snooping_use_range
Conversation
| } | ||
| } | ||
|
|
||
| if ( pOldCont != pItem->GetContainer() ) |
There was a problem hiding this comment.
I remember adding this for a reason, does git blame help for that?
There was a problem hiding this comment.
i blame it and see that doesnt change in years
https://github.com/Sphereserver/Source-X/blame/ae696679c35edce7192b8fbb55350ba28003718d/src/graysvr/CClientEvent.cpp
There was a problem hiding this comment.
This check should be there, because this check if the item changed the container while dropon_item trigger checking;
I mean think like 2 players tries to get same item from the corpse or ground, and you did some checks under dropon_item, for the first player, then added the item in his container, you don't want to second player take it away from player's backpack. This check is added to block this, but it probably could be really really rare issue, as sphere can handle these checks in milliseconds, but as someone can have tons of checks under @DropOn_Item, checking if the old container is same is a guarantee. So it has to stay as it was honestly.
|
Duplication of #1338? |
4 participants
Implement #1142
and minor code clean up of unused vars