This should be a simple fix, there just needs to be a case for skipping the current line if it has zero length or is empty/whitespace.
I suspect there's a larger issue with error handling, as currently the problematic playlist causes the playlist scanner to halt as well, even if there are other valid playlists to scan.
Affected area
[X] Strix Music SDK
[ ] Strix Music App
Regression
No response
Steps to reproduce
1. Create an M3U playlist and add an empty line anywhere in the file
2. Attempt to load said playlist into Strix Music
3. Observe an `IndexOutOfRangeException` in the log output, and the playlist scanner halting
Visual repro steps
No response
Expected behavior
Playlist parses ignores empty lines.
Additional context
No response
Help us help you
Yes, I'd like to be assigned to work on this item.
Describe the bug
M3U playlists containing empty lines throw an
IndexOutOfRangeException
at the following line when attempting to parse. https://github.com/Arlodotexe/strix-music/blob/0bdf1f349be8d726184a686cc43aa2246e4df3ec/src/Sdk/StrixMusic.Sdk/FileMetadata/Scanners/PlaylistMetadataScanner.Parsers.cs#L154This should be a simple fix, there just needs to be a case for skipping the current line if it has zero length or is empty/whitespace.
I suspect there's a larger issue with error handling, as currently the problematic playlist causes the playlist scanner to halt as well, even if there are other valid playlists to scan.
Affected area
Regression
No response
Steps to reproduce
Visual repro steps
No response
Expected behavior
Playlist parses ignores empty lines.
Additional context
No response
Help us help you
Yes, I'd like to be assigned to work on this item.