[opensource-dev] Review Request: VWR-26066: request LLFloaterWorldMap child "zoom slider" with correct type to get rid of warning when opening map flaoter
Boroondas Gupte
sllists at boroon.dasgupta.ch
Mon Jun 20 06:35:20 PDT 2011
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/350/
-----------------------------------------------------------
Review request for Viewer.
Summary
-------
To reproduce
1. Start the viewer from a terminal, or make the debug console visible after logging in
2. Open the map e.g. by pressing Ctrl-m or by clicking the Map button (can be made visible from the bottom bar context menu)
Expected
* Map opens
* Debug console or terminal displays
INFO: openFloater: Opening floater worldmap
(and no warning)
Observed
* Map opens
* Debug console or terminal displays
INFO: openFloater: Opening floater worldmap
WARNING: getChild: Found child named "zoom slider" but of wrong type 12LLSliderCtrl, expecting P8LLSlider
This is due to requesting the child control with the wrong type, which this change fixes.
This addresses bug VWR-26066.
http://jira.secondlife.com/browse/VWR-26066
Diffs
-----
doc/contributions.txt 848ad0e546f8
indra/newview/llfloaterworldmap.cpp 848ad0e546f8
Diff: http://codereview.secondlife.com/r/350/diff
Testing
-------
Merged this fix with e67da2c6e312 and built (linux 64 standalone)
* Warning gone
* Map still works, no perceptible change in behavior noticed.
Not tested: Merging with v-d tip, as I can't build that. (But I know from downloaded test builds that it is affected by this bug.)
Thanks,
Boroondas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.secondlife.com/pipermail/opensource-dev/attachments/20110620/d677f423/attachment.htm
More information about the opensource-dev
mailing list