Fix ArrayIndexOutOfBoundsException with compact SVG arc notation - #1282
Conversation
Sorry, something went wrong.
There was a problem hiding this comment.
Pull Request Overview
This PR fixes an ArrayIndexOutOfBoundsException that occurred when parsing SVG arc commands with compact notation where flags and coordinates are concatenated (e.g., a2 2 0 013 3). The fix adds support for detecting and properly parsing this compact format.
Key changes:
- Added isCompactArcNotation() helper method to detect concatenated arc flags and coordinates
- Modified arc parsing logic for both absolute (A) and relative (a) commands to handle compact notation
- Added comprehensive test coverage for various compact arc notation scenarios
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| core/src/processing/core/PShapeSVG.java | Implements compact arc notation detection and parsing logic |
| core/test/processing/core/PShapeSVGPathTest.java | Adds test cases for compact notation, standard notation, and edge cases |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Sorry, something went wrong.
|
Hi @hxrshxz great work! Thank you! Looking at the tests, the one thing I would say to be missing is a step that actually checks if the parsing occurred correctly by looking at the numbers that the parser spits out, I would say just checking if the shape parses is not enough. |
Sorry, something went wrong.
|
Thanks @Stefterv I have added tests to verify the actual parsed endpoint values, not just that the shape doesn't crash |
Sorry, something went wrong.
catilac
left a comment
There was a problem hiding this comment.
Looks good! I have some questions. Sorry to take so long with this.
Sorry, something went wrong.
|
really sorry for taking so long |
Sorry, something went wrong.
|
@hxrshxz this is approved! yay. we are going to do some additional hand testing before it gets merged in. we have an upcoming 4.5 release so i'm being extra careful. but thank you so much! sorry to take so long in accepting this. |
Sorry, something went wrong.
Fixes #1244
Summary
Adds support for compact SVG arc notation where flags and coordinates are concatenated (e.g., a2 2 0 013 3 instead of a 2 2 0 0 1 3 3). Previously crashed with ArrayIndexOutOfBoundsException.
Changes
Before
Screencast.from.2025-10-15.00-31-18.mp4After
Screencast.from.2025-10-14.23-14-15.mp4