[opensource-dev] Review Request: (STORM-550) LLDir::getNextFileInDir fails for some complex wildcard combinations

Seth ProductEngine slitovchuk at productengine.com
Fri Dec 17 12:46:52 PST 2010


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

Review request for Viewer.


Summary
-------

Fixed LLDir unit test which failed for some complex wildcard combinations.
Added a class implementing directory entries iteration with pattern matching which is used in unit tests instead of LLDir::getNextFileInDir.

This code has been run on Linux only. It should be tested under other platforms and more test cases should be provided. For example changing directory contents while iterating through it.


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


Diffs
-----

  indra/cmake/Boost.cmake 794ad1fc71d1 
  indra/llvfs/CMakeLists.txt 794ad1fc71d1 
  indra/llvfs/lldiriterator.h PRE-CREATION 
  indra/llvfs/lldiriterator.cpp PRE-CREATION 
  indra/llvfs/tests/lldir_test.cpp 794ad1fc71d1 

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


Testing
-------


Thanks,

Seth

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.secondlife.com/pipermail/opensource-dev/attachments/20101217/23eef602/attachment.htm 


More information about the opensource-dev mailing list