Search Unity

Feature Request Add methods to add several Transforms to a TransformAccessArray

Discussion in 'C# Job System' started by Enderlook, Mar 2, 2023.

  1. Enderlook

    Enderlook

    Joined:
    Dec 4, 2018
    Posts:
    50
    The following type contains a constructor with the signature:
    Code (CSharp):
    1. public TransformAccessArray(Transform[] transforms, int desiredJobCount);
    I propose adding the overloads to the constructor:
    Code (CSharp):
    1. public TransformAccessArray(ReadOnlySpan<Transform> transforms, int desiredJobCount);
    2. public TransformAccessArray(List<Transform> transforms, int desiredJobCount);
    And the following overloads to the method:
    Code (CSharp):
    1. public void SetTransforms(ReadOnlySpan<Transform> transforms);
    2. public void SetTransforms(List<Transform> transforms);
    And the new method:
    Code (CSharp):
    1. public void AddRange(Transform[] transforms);
    2. public void AddRange(ReadOnlySpan<Transform> transforms);
    3. public void AddRange(List<Transform> transforms);
    Alternatively, adding overloads that accept count would also be useful such as:
    Code (CSharp):
    1. public void SetTransforms(Transform[] transforms, int count)
    So you can specify how many items you want to set.

    This would make it easier to construct TransformAccessArray with populated results.
    Of course, you can right now emulate them by iterating the collection and using the Add method, or creating a new array with the specific size, but that is more expensive due to potential allocations, safety checks, and/or internal calls.

    Having that additional overloads would make stuff easier and more performant, and they are trivial to implement.
     
    VirtusH, fxlange, forestrf and 3 others like this.
  2. Qriva

    Qriva

    Joined:
    Jun 30, 2019
    Posts:
    1,307
    I still don't understand why literally everything related to job system have overloads for everything, meanwhile TransformAccessArray content manipulation is so poor and mysterious.
     
  3. forestrf

    forestrf

    Joined:
    Aug 28, 2010
    Posts:
    230
    Yes, but doing that would make sense.