[opensource-dev] Review Request: VWR-25965 LLDirIterator fixes in the viewer cache migration and VFS fallback.

[Altair Memo]

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

Ship it!


work fine on linux, all test passed (no problem if cache is stored on ramdisk too, erased every reboot start fine each run)

- Altair


On June 7, 2011, 11:42 a.m., Log Linden wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://codereview.secondlife.com/r/324/
> -----------------------------------------------------------
> 
> (Updated June 7, 2011, 11:42 a.m.)
> 
> 
> Review request for Viewer.
> 
> 
> Summary
> -------
> 
> This removes additional leading delimiters in llappviewer needed due to the STORM-477 changes. This change should fix the fallback used when the viewer cannot find a VFS with the salt it expects to find and instead looks for any VFS data file in the cache directory to use.  It also fixes cache directory migration code that runs on Mac and Windows every time the viewer is updated. 
> 
> 
> This addresses bug VWR-25964.
>     http://jira.secondlife.com/browse/VWR-25964
> 
> 
> Diffs
> -----
> 
>   indra/newview/llappviewer.cpp 1d4d568986e3 
> 
> Diff: http://codereview.secondlife.com/r/324/diff
> 
> 
> Testing
> -------
> 
> Test Plan:
> https://jira.secondlife.com/browse/VWR-25965?focusedCommentId=264386
> 
> I ran Case 1 on Linux and Case 2 on Mac and Windows. Everything worked as expected.
> 
> 
> Thanks,
> 
> Log
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.secondlife.com/pipermail/opensource-dev/attachments/20110607/1d2a6d1d/attachment.htm