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.

Sign up for GitHub

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

Jump to bottom

エンディアンの扱いや設定が不明瞭なので整理する #277

Open

2 tasks

meltingrabbit opened this issue Jan 4, 2024 · 0 comments

Labels

priority::medium

Member

meltingrabbit commented Jan 4, 2024

詳細

C2A 初期化時に実行環境のエンディアンが設定と正しいかチェックする #260 (comment) で発生した
やるべきことは２つ
- src_user/settings/build_settings.h のコメントをあつくするなどし，IS_LITTLE_ENDIAN がリトルエンディアン環境では必須であることを明らかにする
- 抜本的な整理
  - little endian な場合 IS_LITTLE_ENDIAN が define されてないと，C2Aは正しく動かないは明らかでない上に confusing なので，この挙動は変えていい．
  - IS_ みたいな曖昧な分岐ではなく，big or little 両方の場合に必ず define させるとか．
  - IS_LITTLE_ENDIAN については，一旦は IS_BIG_ENDIAN を生やすみたいな対応にすると，breaking change ではなくあくまで仕様追加として行えるかなと思います．

The text was updated successfully, but these errors were encountered:

meltingrabbit added the priority::medium label

meltingrabbit mentioned this issue

C2A 初期化時に実行環境のエンディアンが設定と正しいかチェックする #260

Merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment