c2a-core icon indicating copy to clipboard operation
c2a-core copied to clipboard

CCSDS関連のコードを全部書き直したい

Open meltingrabbit opened this issue 2 years ago • 0 comments

概要

CCSDS関連のコードを全部書き直したい

詳細

以下などのように,全体的にレガシーコードがおおいいので,直したい.

  • こういう部分がシリアライズしにくい(構造体パディングが環境依存なのに考慮されてない)
    • https://github.com/ut-issl/c2a-core/blob/578c6e529f677696f4b7cb932da5b9b99db44d4e/Examples/minimum_user_for_s2e/src/src_user/TlmCmd/Ccsds/TCFrame.h#L17-L22
  • こことかが,上の話を考えると不適切.また,body部分が可変長なので,footer位置が構造体定義とずれてる
    • https://github.com/ut-issl/c2a-core/blob/578c6e529f677696f4b7cb932da5b9b99db44d4e/Examples/minimum_user_for_s2e/src/src_user/Drivers/Com/gs.c#L212
  • 構造体位置がずれてるので,こことかでは,わざわざlenをたわしてる(不適切)
    • https://github.com/ut-issl/c2a-core/blob/578c6e529f677696f4b7cb932da5b9b99db44d4e/Examples/minimum_user_for_s2e/src/src_user/Drivers/Com/gs_validate.c#L140
  • CRCのライブラリを使うべし
    • https://github.com/ut-issl/c2a-core/blob/578c6e529f677696f4b7cb932da5b9b99db44d4e/Examples/minimum_user_for_s2e/src/src_user/Drivers/Com/gs_validate.c#L173-L174

close条件

よくなったら

関連

  • https://github.com/ut-issl/c2a-core/issues/198

meltingrabbit avatar Mar 07 '22 09:03 meltingrabbit