----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/93/#review224 -----------------------------------------------------------
indra/newview/lltexturecache.cpp <http://codereview.secondlife.com/r/93/#comment154> I get it but it's a bit unsettling to have the condition seemingly unrelated to the iterator. I'd rather have a while () construction here so it's more clear what the intent is. - Merov On Jan. 14, 2011, 1:09 p.m., Aleric Inglewood wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/93/ > ----------------------------------------------------------- > > (Updated Jan. 14, 2011, 1:09 p.m.) > > > Review request for Viewer. > > > Summary > ------- > > Just fixed the logic, so entries_to_purge won't become negative anymore, and > the rest. > > > This addresses bug VWR-24337. > http://jira.secondlife.com/browse/VWR-24337 > > > Diffs > ----- > > doc/contributions.txt b0bd26c5638a > indra/newview/lltexturecache.cpp b0bd26c5638a > > Diff: http://codereview.secondlife.com/r/93/diff > > > Testing > ------- > > > 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
