[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