LL perf (Inventory): `collectDescendents` used to count or to find links #347

New issue

Closed

opened 2026-10-04 14:09:25 +00:00 by sky_line · 1 comment

sky_line commented

2026-10-04 14:09:25 +00:00

Owner

From the LL-origin inefficiency audit (finding I11). File/line references are from commit 0229f8872c; re-locate them before working.

collectDescendents used to count or to find links

Where: LLUpdateFavorite::fire walks the entire inventory per favorite toggle (llinventoryfunctions.cpp:2533); also llinventorypanel.cpp:1895, llviewerinventory.cpp:2408, 2433, llappearancemgr.cpp:3261.
Fix: collectLinksTo, counting functors.
Confirmed. S.

Verify the cost (profile or timer) before and after changing it.

From the LL-origin inefficiency audit (finding I11). File/line references are from commit 0229f8872c; re-locate them before working. **`collectDescendents` used to count or to find links** - **Where:** `LLUpdateFavorite::fire` walks the **entire inventory** per favorite toggle (`llinventoryfunctions.cpp:2533`); also `llinventorypanel.cpp:1895`, `llviewerinventory.cpp:2408, 2433`, `llappearancemgr.cpp:3261`. - **Fix:** `collectLinksTo`, counting functors. - Confirmed. S. Verify the cost (profile or timer) before and after changing it.

sky_line added the

perf

label

2026-10-04 14:09:25 +00:00

sky_line added this to the Slipstream project

2026-10-06 15:04:13 +00:00

sky_line added the

status

in-progress

label

2026-10-08 02:07:13 +00:00

sky_line referenced this issue from a commit