[sldev] [VIEWER] Dead code removal

[Kelly Linden]

Lawson English wrote:
> Bryan O'Sullivan wrote:
>> Dirk Moerenhout wrote:
>>> True but I'm mostly referring to parts of newview and I hope it is at
>>> least so that this is never used by the server code.
>>
>> It's best to keep the dead code removal in a patch of its own.  That
>> makes it easier for us to review patches.
>>
>>> It might even be
>>> a good question: What do we really share with the server code? I guess
>>> we probably share llcommon but what other parts are dangerous too?
>>
>> You can see the exact lists of libraries that the viewer and server
>> depend on in the SConstruct file that ships with the source.
>>
>> A lot of the libraries are shared.  The easiest rule of thumb is to
>> assume that any ll* directory that isn't related to drawing windows
>> or other bits of UI is likely to be shared with the server code.
>
> I'm confused... The client build ships with server-only  libraries
> included?
>
>
No, but some of the libraries used by the client are also used by the
servers.  This means that some code in the libraries that appears to be
'dead' from the viewer perspective may not be, if any of the server code
relies on it.

 - Kelly