Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix playWav failed if exists some chunks between fmtChunk and dataChunk #30

Merged
merged 2 commits into from
Jun 7, 2022
Merged

Fix playWav failed if exists some chunks between fmtChunk and dataChunk #30

merged 2 commits into from
Jun 7, 2022

Conversation

GOB52
Copy link
Contributor

@GOB52 GOB52 commented Jun 6, 2022

Speaker_Class::playWav

fmtChunk と dataChunk の間に別のチャンクが存在すると再生に失敗します。
dataChunk まで読み飛ばす処理を追加。
playWav failed if exists some chunks between fmtChunk and dataChunk.
Add code to skip until dataChunk.

データ側でそもそも fmtChunk の次が dataChunk の物を作ればコードの変更は必要ありませんが、現状 FFMPEG 他のコンバータや出力ツールでは INFO や LIST のチャンクが維持/付与される場合が多く、開発環境上のツールでは一見正しいリソースで鳴動しているのになんで実機だとならないんだというハマリポイントになりかねない気がします。
よろしくご検討ください。

@lovyan03
Copy link
Collaborator

lovyan03 commented Jun 6, 2022

プルリクエストありがとうございます!助かります。
マージさせて頂きたいと思いますが、ひとまずdevelopブランチに入れたいです。
お手数ですが、マージ先の変更をお願いしても良いでしょうか?

@GOB52 GOB52 changed the base branch from master to develop June 7, 2022 04:51
@lovyan03 lovyan03 merged commit 0ffbe24 into m5stack:develop Jun 7, 2022
@GOB52 GOB52 deleted the feature/speaker-playWav branch June 8, 2022 07:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants