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

Play_Sound: Fix loading WAV file on big endian machine #725

Merged

Conversation

ma8ma
Copy link
Collaborator

@ma8ma ma8ma commented May 15, 2021

RIFFフォーマットのマルチバイトデータはリトルエンディアンで保存されるためビッグエンディアンの環境ではストリームから読み込んだデータのバイトオーダーを逆順に並び替えます。

  • RIFFチャンクのデータサイズを明示的に指定するためWAVデータの格納に固定長の整数型を使用します。
  • Autotoolsのバージョン要件を2.69に更新します。
  • configure.acのAC_ISC_POSIXマクロは廃止予定となったため削除します。

ma8ma added 2 commits May 15, 2021 19:56
RIFFチャンクのデータサイズを明示的に指定するためWAVデータの格納に
固定長の整数型を使用します。
RIFFフォーマットのマルチバイトデータはリトルエンディアンで
保存されるためビッグエンディアンの環境ではストリームから読み込んだ
データのバイトオーダーを逆順に並び替えます。

- Autotoolsのバージョン要件を2.69に更新します。
- `AC_ISC_POSIX`マクロは廃止予定[1]となったため削除します。

[1]: https://www.gnu.org/software/autoconf/manual/autoconf-2.61/html_node/Posix-Variants.html
@ma8ma ma8ma added the bug バグの追跡 label May 15, 2021
@ma8ma ma8ma added this to In progress in State of JDim-v0.5.0+ via automation May 15, 2021
@ma8ma ma8ma merged commit 4bde419 into JDimproved:master May 15, 2021
State of JDim-v0.5.0+ automation moved this from In progress to Done May 15, 2021
@ma8ma ma8ma deleted the playsound-fix-load-wavfile-on-big-endian branch May 15, 2021 11:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug バグの追跡
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

1 participant