refactor: In llRemoveInventory() use existing GetInventoryItem() method rather than have it iterate through TaskInventory itself.

0.7.3-extended
Justin Clark-Casey (justincc) 2012-07-04 21:05:51 +01:00
parent 218fe36f84
commit 9a0bcb7750
1 changed files with 9 additions and 14 deletions

View File

@ -3862,20 +3862,15 @@ namespace OpenSim.Region.ScriptEngine.Shared.Api
{ {
m_host.AddScriptLPS(1); m_host.AddScriptLPS(1);
lock (m_host.TaskInventory) TaskInventoryItem item = m_host.Inventory.GetInventoryItem(name);
{
foreach (TaskInventoryItem item in m_host.TaskInventory.Values) if (item == null)
{ return;
if (item.Name == name)
{
if (item.ItemID == m_item.ItemID) if (item.ItemID == m_item.ItemID)
throw new ScriptDeleteException(); throw new ScriptDeleteException();
else else
m_host.Inventory.RemoveInventoryItem(item.ItemID); m_host.Inventory.RemoveInventoryItem(item.ItemID);
return;
}
}
}
} }
public void llSetText(string text, LSL_Vector color, double alpha) public void llSetText(string text, LSL_Vector color, double alpha)