fixed computedFn()'s to be used correctly in several places. fixed major memory leak in PDFs

1 files changed, 2 insertions, 3 deletions

diff --git a/src/client/util/SnappingManager.ts b/src/client/util/SnappingManager.ts
index a615f0247..069f81d38 100644
--- a/src/client/util/SnappingManager.ts
+++ b/src/client/util/SnappingManager.ts
@@ -32,9 +32,8 @@ export namespace SnappingManager {
 
     /// bcz; argh!! TODO;   These do not belong here, but there were include order problems with leaving them in util.ts
     // need to investigate further what caused the mobx update problems and move to a better location.
+    const getCachedGroupByNameCache = computedFn(function (name: string) { return manager.cachedGroups.includes(name); }, true);
+    export function GetCachedGroupByName(name: string) { return getCachedGroupByNameCache(name); }
     export function SetCachedGroups(groups: string[]) { manager.setCachedGroups(groups); }
-    export function GetCachedGroupByName(name: string) {
-        return computedFn(function (name: string) { return manager.cachedGroups.includes(name); }, true)(name);
-    }
 }