-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/261/
-----------------------------------------------------------

(Updated April 11, 2011, 9:27 a.m.)


Review request for Viewer.


Summary (updated)
-------

Fixed the inventory observers of newly added items.

The problem was caused by an outdated message name stored in 
LLInventoryObserver::mMessageName and not updated properly in 
LLInventoryModel::notifyObservers(). 
The message name used in LLInventoryAddedObserver::changed() was the name of 
the message most recently passed by LLInventoryModel::notifyObservers(), 
instead of the name of the latest actually received message. Using the most 
recent message name in this case fixed the problem.


This addresses bug STORM-1042.
    http://jira.secondlife.com/browse/STORM-1042


Diffs
-----

  indra/newview/llinventorymodel.h d30636c2a83a 
  indra/newview/llinventorymodel.cpp d30636c2a83a 
  indra/newview/llinventorymodelbackgroundfetch.cpp d30636c2a83a 
  indra/newview/llinventoryobserver.h d30636c2a83a 
  indra/newview/llinventoryobserver.cpp d30636c2a83a 

Diff: http://codereview.secondlife.com/r/261/diff


Testing
-------


Thanks,

Seth

_______________________________________________
Policies and (un)subscribe information available here:
http://wiki.secondlife.com/wiki/OpenSource-Dev
Please read the policies before posting to keep unmoderated posting privileges

Reply via email to