by Graf Zahl » Sat Feb 05, 2005 5:16 pm
Well, the removal of the item in APuzzleItem::Use is redundant because it gets done again in AActor::UseInventory - even if it is used by activating a line special.
While we are discussing puzzle items there is one other thing: If the puzzle item has to be used on a line you can activate the line by the specified means but when you have to use it on a thing you have to use the item in the inventory. But the code to handle 'using' Strife's characters to start a conversation is already there so why not using it to check whether a puzzle item can be used on a thing that's in activation range?
Well, the removal of the item in APuzzleItem::Use is redundant because it gets done again in AActor::UseInventory - even if it is used by activating a line special.
While we are discussing puzzle items there is one other thing: If the puzzle item has to be used on a line you can activate the line by the specified means but when you have to use it on a thing you have to use the item in the inventory. But the code to handle 'using' Strife's characters to start a conversation is already there so why not using it to check whether a puzzle item can be used on a thing that's in activation range?