-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/451/
-----------------------------------------------------------
(Updated Aug. 27, 2011, 3:01 a.m.)
Review request for Viewer and Seth ProductEngine.
Changes
-------
Fixed a silly pointer usage mistake.
Summary
-------
Reason: Showing the confirmation dialog resets object selection,
thus there is nothing to derez.
Fix: Save selection until user answers in the confirmation dialog.
I didn't investigate why the bug occurred only for distant object
(must be some internal LLSelectMgr magic).
This addresses bug STORM-638.
http://jira.secondlife.com/browse/STORM-638
Diffs (updated)
-----
indra/newview/llviewermenu.cpp 00b00b942134
Diff: http://codereview.secondlife.com/r/451/diff
Testing
-------
See acceptance criteria in the JIRA ticket.
Thanks,
Vadim
_______________________________________________
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
