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

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

Merged
merged 4 commits into from
Jan 5, 2024

Conversation

meltingrabbit
Copy link
Member

@meltingrabbit meltingrabbit commented Dec 26, 2023

概要

C2A 初期化時に実行環境のエンディアンが設定と正しいかチェックする.
初期化中なので,警告は出すが,処理は止めない.

これによって,C2A移植時のミスが減る(このミスによるつまずきが多々ある)

Issue

検証結果

build and exec CI が通ればOK

image

@meltingrabbit meltingrabbit added enhancement New feature or request priority::medium priority medium labels Dec 26, 2023
@meltingrabbit meltingrabbit self-assigned this Dec 26, 2023
library/endian.c Show resolved Hide resolved
c2a_core_main.c Show resolved Hide resolved
@meltingrabbit
Copy link
Member Author

コンフリクトしてるので #275 がマージされたらマージする

@meltingrabbit meltingrabbit merged commit 477545b into main Jan 5, 2024
37 checks passed
@meltingrabbit meltingrabbit deleted the feature/endian-detection branch January 5, 2024 16:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request priority::medium priority medium
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants