_bindBuffer unnecessarily copies typed arrays
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 24k
- Forks
- 3.8k
- Avg merge
- 3d 16h
- Merged PRs (30d)
- 25
Description
Most appropriate sub-area of p5.js?
- Accessibility
- Color
- Core/Environment/Rendering
- Data
- DOM
- Events
- Image
- IO
- Math
- Typography
- Utilities
- WebGL
- Build Process
- Unit Testing
- Localization Tools
- Friendly Errors
- Other (specify if possible)
p5.js version
main
Web browser and version
No response
Operating System
No response
Steps to reproduce this
I documented the behaviour in this PR that got closed because of some process you prefer. https://github.com/processing/p5.js/pull/5737
I don't have a lot of spare time, and while I'll happily do value-adding things to allow this to be merged, I have no plans to become a long term contributor, so someone with full awareness of the hoops you've got present might want to take this over.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reviewing the behavior documented in PR #5737 and locate the _bindBuffer entry point in p5.js. Confirm how typed arrays are handled and identify the expected behavior from that discussion; done means the unnecessary copying behavior is addressed and the relevant regression coverage passes.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- performance
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 30/100