Speed up building the dictionary of reused Elm value encodings

pine-vm · Oct 5, 2024 · c95afbe · c95afbe
1 parent 97f67f8
commit c95afbe
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/implement/Pine.Core/ReusedInstances.cs b/implement/Pine.Core/ReusedInstances.cs
@@ -343,9 +343,15 @@ [.. elmRecord.Fields.Select(kvp => (kvp.FieldName, reusedInstances[kvp.Value]))]
         {
             var encodedDict = new Dictionary<ElmValue, PineValue>();
 
-            foreach (var item in ElmValues)
+            var elmValuesSorted =
+                ElmValues
+                .OrderBy(ev => ev.ContainedNodesCount)
+                .ToList();
+
+            foreach (var item in elmValuesSorted)
             {
-                encodedDict[item] = ElmInteractive.ElmValueEncoding.ElmValueAsPineValue(item);
+                encodedDict[item] =
+                    ElmInteractive.ElmValueEncoding.ElmValueAsPineValue(item, encodedDict);
             }
 
             ElmValueEncoding =