stride
stride copied to clipboard
feat: Deprecate DataPointer in favor of Span
PR Details
DataPointer are just the more rigid and error prone version of what Span provides, marked that type as deprecated, created new signatures providing support for spans instead and moved over the engine calls to that one instead.
One thing I tried to tackle is replacing T[] with Span<T> to reduce the ridiculous amount of signatures Buffer and its variants already hosts, but arrays do not implicitly convert to spans when the function they are passed into is generic. Replacing all cases of T[] with Span<T> would fail to find the method. We'll have to mark those methods as obsolete as well if we want to clean that up.
Related Issue
None
Types of changes
- [ ] Docs change / refactoring / dependency upgrade
- [ ] Bug fix (non-breaking change which fixes an issue)
- [x] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to change)
Checklist
- [ ] My change requires a change to the documentation.
- [ ] I have added tests to cover my changes.
- [ ] All new and existing tests passed.
- [x] I have built and run the editor to try this change out.