----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/63/#review79 -----------------------------------------------------------
Ship it! No reason not to submit. :-) indra/llkdu/llimagej2ckdu.h <http://codereview.secondlife.com/r/63/#comment46> Please add a comment that these methods aren't actually public, i.e. were made public only to be called from unit tests. indra/llkdu/tests/llimagej2ckdu_test.cpp <http://codereview.secondlife.com/r/63/#comment47> What about calling protected methods via inheritance? indra/llkdu/tests/llimagej2ckdu_test.cpp <http://codereview.secondlife.com/r/63/#comment48> I suppose unit tests should verify something more substantial than not raising exceptions for empty data. - Vadim On 2010-12-22 23:51:18, Merov Linden wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/63/ > ----------------------------------------------------------- > > (Updated 2010-12-22 23:51:18) > > > Review request for Viewer. > > > Summary > ------- > > Unit tests addition: > - add tests for llkdu > - turned back on and fix unit tests for llimage > - turned back on and fix unit tests for llworldmap and llworldmipmap > > > This addresses bug STORM-744. > http://jira.secondlife.com/browse/STORM-744 > > > Diffs > ----- > > indra/llimage/CMakeLists.txt 5d69e36a53ee > indra/llimage/tests/llimageworker_test.cpp 5d69e36a53ee > indra/llkdu/CMakeLists.txt 5d69e36a53ee > indra/llkdu/llimagej2ckdu.h 5d69e36a53ee > indra/llkdu/tests/llimagej2ckdu_test.cpp PRE-CREATION > indra/newview/CMakeLists.txt 5d69e36a53ee > indra/newview/tests/llworldmap_test.cpp 5d69e36a53ee > indra/newview/tests/llworldmipmap_test.cpp 5d69e36a53ee > > Diff: http://codereview.secondlife.com/r/63/diff > > > Testing > ------- > > > Thanks, > > Merov > >
_______________________________________________ Policies and (un)subscribe information available here: http://wiki.secondlife.com/wiki/OpenSource-Dev Please read the policies before posting to keep unmoderated posting privileges
