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
Owner

@theovilardo theovilardo commented Nov 2, 2025

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

codex

1 participant

@theovilardo