-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/503/
-----------------------------------------------------------
Review request for Viewer.
Summary
-------
The implementation in the viewer of the local wind eddies was too closely
coupled to that of the Classic Clouds, so that when the clouds were removed in
v2.8 the local wind eddies were broken.
This change removes that coupling (along with a few vestigial bits of the
classic clouds) and restores the wind eddies.
It turned out that there was also some debug code for visualizing the wind, so
I attached that to the Develop>Render Metadata menu (Wind Vectors), adjusting
it so that it displays the vectors 25 meters above the avatar (attaching
screenshots to the jira issue). Thanks are due to Runitai Linden for help with
this part.
This addresses bug storm-1622.
http://jira.secondlife.com/browse/storm-1622
Diffs
-----
indra/newview/llglsandbox.cpp 913d90c3225b
indra/newview/llviewermenu.cpp 913d90c3225b
indra/newview/llwind.h 913d90c3225b
indra/newview/llwind.cpp 913d90c3225b
indra/newview/pipeline.h 913d90c3225b
indra/newview/pipeline.cpp 913d90c3225b
indra/newview/skins/default/xui/en/menu_viewer.xml 913d90c3225b
Diff: http://codereview.secondlife.com/r/503/diff
Testing
-------
Tested with particles that are sensitive to wind, and the excellent
demonstration site linked in the jira issue.
Thanks,
Oz
_______________________________________________
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