Skip to content

Rework combine #43

Description

@kMutagene

We need to find a way to perform a deep copy before combining. That way, most mutation-related issues are solved.

Continuation of #35 , which originally aimed to canonize Combine and Copy. However, these functions are slightly different from each other, and Combine needs some more work. Notably, it was possible to provide a combine function that keeps a derived type inheriting from DynamicObj. This will be lost when performing a deep copy, as that returns a boxed DynamicObj.

I think we need DeepCombine and ShallowCombine respectively, maybe some API that also helps with the subsequent unboxing, e.g. DeepCombineAs<'T>

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions