[opensource-dev] Review Request: STORM-64: Local Bitmaps 2.0 implementation.

[Vaalith Jinn]

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

(Updated Sept. 4, 2011, 8:17 a.m.)


Review request for Viewer.


Summary (updated)
-------

Local Bitmaps is a mechanism to locally load images into the viewer, track them and optionally (per each image)
have it check if the image has been overwritten locally and if so - update it in the viewer and inworld.

This change affects the texture picker - adding radio checks that let the user choose between the "Inventory" (regular inventory) and "Local" tabs (list of locally added files).


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


Diffs (updated)
-----

  doc/contributions.txt 8da01486a36a 
  indra/newview/CMakeLists.txt 8da01486a36a 
  indra/newview/lllocalbitmaps.h PRE-CREATION 
  indra/newview/lllocalbitmaps.cpp PRE-CREATION 
  indra/newview/lltexturectrl.h 8da01486a36a 
  indra/newview/lltexturectrl.cpp 8da01486a36a 
  indra/newview/llviewertexturelist.h 8da01486a36a 
  indra/newview/llwearable.h 8da01486a36a 
  indra/newview/llwearable.cpp 8da01486a36a 
  indra/newview/skins/default/xui/en/floater_texture_ctrl.xml 8da01486a36a 

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


Testing (updated)
-------

Texture/Sculptmap/Avatar Layer show.
Texture/Sculptmap/Avatar Layer update.
Multiple sculpties update.

* Only tested this myself, as opposed to the previous implementation which has been live for, uh, a year,
this one is completely new, written mostly from scratch and uses different mechanisms


Thanks,

Vaalith

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.secondlife.com/pipermail/opensource-dev/attachments/20110904/00edd5a7/attachment.htm