[opensource-dev] Review Request: Implementation of new testing hooks to enable the creation of a headless viewer for testing.

Boroondas Gupte sllists at boroon.dasgupta.ch
Mon Mar 7 15:40:44 PST 2011


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


Any chance this big change could be split into several smaller (hopefully self-consistent) changes that could be reviewed separately? Dunno if that's feasible. If not, please at least push the hg changesets included in this change to some publicly readable repo, so we can look at those individually.

- Boroondas


On March 7, 2011, 3:22 p.m., Merov Linden wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://codereview.secondlife.com/r/181/
> -----------------------------------------------------------
> 
> (Updated March 7, 2011, 3:22 p.m.)
> 
> 
> Review request for Viewer and Oz Linden.
> 
> 
> Summary
> -------
> 
> Massive changes implementing a set of new testing hooks so to enable the creation of a headless viewer for testing.
> 
> 
> This addresses bug STORM-1051.
>     http://jira.secondlife.com/browse/STORM-1051
> 
> 
> Diffs
> -----
> 
>   .hgtags 8b9f50878cc7 
>   indra/llcommon/CMakeLists.txt 8b9f50878cc7 
>   indra/llcommon/lleventdispatcher.h 8b9f50878cc7 
>   indra/llcommon/lleventdispatcher.cpp 8b9f50878cc7 
>   indra/llcommon/llevents.h 8b9f50878cc7 
>   indra/llcommon/llevents.cpp 8b9f50878cc7 
>   indra/llcommon/llsdutil.h 8b9f50878cc7 
>   indra/llcommon/llsdutil.cpp 8b9f50878cc7 
>   indra/llcommon/tests/lleventdispatcher_test.cpp PRE-CREATION 
>   indra/llrender/llgl.h 8b9f50878cc7 
>   indra/llrender/llgl.cpp 8b9f50878cc7 
>   indra/llrender/llimagegl.cpp 8b9f50878cc7 
>   indra/llui/llfloaterreglistener.h 8b9f50878cc7 
>   indra/llui/llfloaterreglistener.cpp 8b9f50878cc7 
>   indra/llui/llui.h 8b9f50878cc7 
>   indra/llui/llui.cpp 8b9f50878cc7 
>   indra/llui/tests/llurlentry_stub.cpp 8b9f50878cc7 
>   indra/llwindow/CMakeLists.txt 8b9f50878cc7 
>   indra/llwindow/llkeyboardheadless.h PRE-CREATION 
>   indra/llwindow/llkeyboardheadless.cpp PRE-CREATION 
>   indra/llwindow/llwindow.h 8b9f50878cc7 
>   indra/llwindow/llwindow.cpp 8b9f50878cc7 
>   indra/llwindow/llwindowheadless.cpp 8b9f50878cc7 
>   indra/llwindow/llwindowlistener.h PRE-CREATION 
>   indra/llwindow/llwindowlistener.cpp PRE-CREATION 
>   indra/newview/CMakeLists.txt 8b9f50878cc7 
>   indra/newview/app_settings/settings.xml 8b9f50878cc7 
>   indra/newview/llagent.cpp 8b9f50878cc7 
>   indra/newview/llagentcamera.cpp 8b9f50878cc7 
>   indra/newview/llagentlistener.h 8b9f50878cc7 
>   indra/newview/llagentlistener.cpp 8b9f50878cc7 
>   indra/newview/llappviewer.cpp 8b9f50878cc7 
>   indra/newview/llfloaterbump.cpp 8b9f50878cc7 
>   indra/newview/llhudeffectlookat.cpp 8b9f50878cc7 
>   indra/newview/llhudmanager.cpp 8b9f50878cc7 
>   indra/newview/llimview.cpp 8b9f50878cc7 
>   indra/newview/lllogininstance.cpp 8b9f50878cc7 
>   indra/newview/llselectmgr.cpp 8b9f50878cc7 
>   indra/newview/llsidetray.h 8b9f50878cc7 
>   indra/newview/llsidetray.cpp 8b9f50878cc7 
>   indra/newview/llsidetraylistener.h PRE-CREATION 
>   indra/newview/llsidetraylistener.cpp PRE-CREATION 
>   indra/newview/llstartup.cpp 8b9f50878cc7 
>   indra/newview/llsurface.cpp 8b9f50878cc7 
>   indra/newview/lltexturestats.cpp 8b9f50878cc7 
>   indra/newview/lluilistener.h 8b9f50878cc7 
>   indra/newview/lluilistener.cpp 8b9f50878cc7 
>   indra/newview/llviewerdisplay.cpp 8b9f50878cc7 
>   indra/newview/llviewermessage.cpp 8b9f50878cc7 
>   indra/newview/llviewerobject.cpp 8b9f50878cc7 
>   indra/newview/llviewerobjectlist.cpp 8b9f50878cc7 
>   indra/newview/llviewerparcelmgr.cpp 8b9f50878cc7 
>   indra/newview/llviewerregion.cpp 8b9f50878cc7 
>   indra/newview/llviewerstats.cpp 8b9f50878cc7 
>   indra/newview/llviewertexture.cpp 8b9f50878cc7 
>   indra/newview/llviewertexturelist.cpp 8b9f50878cc7 
>   indra/newview/llviewerwindow.cpp 8b9f50878cc7 
>   indra/newview/llviewerwindowlistener.cpp 8b9f50878cc7 
>   indra/newview/llvoavatar.cpp 8b9f50878cc7 
>   indra/newview/llvoavatarself.cpp 8b9f50878cc7 
>   indra/newview/llworld.cpp 8b9f50878cc7 
>   indra/newview/pipeline.cpp 8b9f50878cc7 
> 
> Diff: http://codereview.secondlife.com/r/181/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Merov
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.secondlife.com/pipermail/opensource-dev/attachments/20110307/2fed4dd3/attachment.htm 


More information about the opensource-dev mailing list