Skip to content

Fix parsing of LRC files with BOM characters#345

Merged
theovilardo merged 4 commits intomasterfrom
codex/fix-automatic-synchronization-for-lrc-file
Nov 5, 2025
Merged

Fix parsing of LRC files with BOM characters#345
theovilardo merged 4 commits intomasterfrom
codex/fix-automatic-synchronization-for-lrc-file

Conversation

@theovilardo
Copy link
Copy Markdown
Owner

Summary

  • strip leading byte-order-mark characters from imported LRC lines before matching timestamps
  • ensure the parser continues to populate plain lyrics with sanitized content
  • add a unit test covering LRC input that starts with a BOM so the first timestamp is recognized

Testing

  • ./gradlew testDebugUnitTest (fails: Android SDK is not available in the environment)

https://chatgpt.com/codex/tasks/task_e_69077b67347c832f897196ce345d8a37

@theovilardo theovilardo merged commit 6a8f23d into master Nov 5, 2025
@lostf1sh lostf1sh deleted the codex/fix-automatic-synchronization-for-lrc-file branch January 16, 2026 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant