----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/81/#review153 -----------------------------------------------------------
indra/llcharacter/llanimationstates.cpp <http://codereview.secondlife.com/r/81/#comment95> If all these lines are touched anyway, (didn't select all, to avoid an unnecessary long review), please either remove the alignment or use spaces instead of tabs for aligning, so they will look nice independent of tab display length. indra/llcommon/lllslconstants.h <http://codereview.secondlife.com/r/81/#comment96> Yay for having type modifiers after the core type name. Makes them much easier to understand, especially when there are several cascaded ones. :-) - Boroondas On Jan. 14, 2011, 12:34 p.m., Aleric Inglewood wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/81/ > ----------------------------------------------------------- > > (Updated Jan. 14, 2011, 12:34 p.m.) > > > Review request for Viewer. > > > Summary > ------- > > Turns out that most of my SNOW-800 patch was included in Viewer 2 (albeit > without crediting me). > However, not everything was used and some more cleaning up was possible. > > After this patch, and when compiling with optimization, there are no > duplicates left > anymore that shouldn't be there in the first place: apart from the debug > stream > iostream guard variable, there are several static variables with the same > name (r, r1, > r2, etc) but that indeed actually different symbol objects. Then there are a > few > constant POD arrays that are duplicated a hand full of times because they are > accessed with a variable index (so optimizing them away is not possible). I > left them > like that (although defining those as extern as well would have been more > consistent > and not slower; in fact it would be faster theoretically because those arrays > could > share the same cache page then). > > > This addresses bug VWR-24312. > http://jira.secondlife.com/browse/VWR-24312 > > > Diffs > ----- > > doc/contributions.txt b0bd26c5638a > indra/llcharacter/llanimationstates.cpp b0bd26c5638a > indra/llcommon/llavatarconstants.h b0bd26c5638a > indra/llcommon/lllslconstants.h b0bd26c5638a > indra/llcommon/llmetricperformancetester.h b0bd26c5638a > indra/llmath/llcamera.h b0bd26c5638a > indra/llmath/llcamera.cpp b0bd26c5638a > indra/newview/llviewerobject.cpp b0bd26c5638a > indra/newview/llvoavatar.cpp b0bd26c5638a > indra/newview/llvosky.h b0bd26c5638a > indra/newview/llvosky.cpp b0bd26c5638a > > Diff: http://codereview.secondlife.com/r/81/diff > > > Testing > ------- > > Compiled with optimization and then running readelf on the executable to find > duplicated symbols. > > > Thanks, > > Aleric > >
_______________________________________________ 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
