search

ThePhrozenKeep / D2MOO

Reimplementation of the Diablo2 game coupled with patching facilities for modders.
MIT License
93 stars 29 forks source link

Reconsider `*TradeInventory` names #166

Open AudriusButkevicius opened 2 months ago

AudriusButkevicius commented 2 months ago

They are not really related to trade, so perhaps just removing the word Trade is enough.

Necrolis commented 2 months ago

A lot of the functions marked as "Trade" are actually for the item command queue, and especially the TradeNodes, are just items with pending commands in the inventory.