You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Describe the bug
Much like #603 , SourceNode creates a new instance of a Lazy<AnnotationList> for each and every node in the SourceNode. Which can be many when dealing with big resources.
Expected behavior
We should not allocate a fresh instance of Lazy<T> but use the static Lazy.EnsureInitialized instead:
Describe the bug
Much like #603 ,
SourceNode
creates a new instance of aLazy<AnnotationList>
for each and every node in the SourceNode. Which can be many when dealing with big resources.Expected behavior
We should not allocate a fresh instance of
Lazy<T>
but use the staticLazy.EnsureInitialized
instead:In DomNode.cs:
should be changed to:
This might be worth doing with the
ChildList
property too (in the same file):Additional context
Reported by the FirelyServer team, marked as Firely Favorite.
The text was updated successfully, but these errors were encountered: