[opensource-dev] Review Request: STORM-987 : llimage_libtest : create an independent executable to exercise llimage outside the viewer

Merov Linden merov at lindenlab.com
Wed Mar 9 21:43:37 PST 2011


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

Review request for Viewer.


Summary
-------

This creates a new independent executable that exercise the entire llimage libray, loading and saving all supported image file formats, gathering perf data and outputting basic image info.

The use of the application itself is documented in the wiki: https://wiki.secondlife.com/wiki/Llimage_libtest

>From a code standpoint, the main things done are:
- Add a new test application in indra/integration_tests/llimage_libtest/
- Move some perf amalyzing code from indra/newview/llfasttimerview.cpp to indra/llcommon/llmetricperformancetester so that it can be used outside the newview context (as it should...)
- Small cleanup in llimage.h


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


Diffs
-----

  indra/integration_tests/CMakeLists.txt aed94e854443 
  indra/integration_tests/llimage_libtest/CMakeLists.txt PRE-CREATION 
  indra/integration_tests/llimage_libtest/llimage_libtest.h PRE-CREATION 
  indra/integration_tests/llimage_libtest/llimage_libtest.cpp PRE-CREATION 
  indra/llcommon/llmetricperformancetester.h aed94e854443 
  indra/llcommon/llmetricperformancetester.cpp aed94e854443 
  indra/llimage/llimage.h aed94e854443 
  indra/newview/llfasttimerview.h aed94e854443 
  indra/newview/llfasttimerview.cpp aed94e854443 

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


Testing
-------

So far, testing has been done on Mac only. Tests on Windows and Linux are planned and will likely result in cmake tweaks here and there. We're not planning however to alter dramatically this code further at this point.


Thanks,

Merov

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.secondlife.com/pipermail/opensource-dev/attachments/20110310/15017f96/attachment.htm 


More information about the opensource-dev mailing list