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

未使用変数の削除 #70

Open
suzuki-toshihir0 opened this issue May 29, 2023 · 4 comments
Open

未使用変数の削除 #70

suzuki-toshihir0 opened this issue May 29, 2023 · 4 comments
Labels
🚗 priority::low priority low 🐟 patch Patch Update

Comments

@suzuki-toshihir0
Copy link
Member

概要

未使用変数があり,コンパイラでワーニングが出ているとのことなので対応する.

詳細

別リポジトリではコンパイラでワーニングをCIで落としており,そこで未使用変数が問題となっていた.
該当箇所は以下:

double reject_threshold_set = 0.0;
double amplitude_limit_to_accept_as_step_set = 0.0;

C2A_MATH_ERROR ignored_warning = VECTOR3_normalize(axis_normalized, axis);

C2A_MATH_ERROR ignored_warning = MATRIX33_normalize(matrix_n, matrix);

C2A_MATH_ERROR err_check =

C2A_MATH_ERROR err_check =

C2A_MATH_ERROR err_check =

C2A_MATH_ERROR err_check = VECTOR3_normalize(axis, axis_unnormalized);

C2A_MATH_ERROR dummy = MATRIX33_normalize(matrix, matrix_before_normalize);

C2A_MATH_ERROR dummy_code = QUATERNION_normalize_with_error_check(&q_out, q_in);

ワーニングをCIで落とすようになったら対応が必要になる.

close条件

上記の未使用変数について,以下のどちらかの対策を取ったら

  • 未使用変数の定義をしないようにする
  • 明示的にunusedする

備考

なにかあれば

注意

  • 関連する Projects が存在する場合,それの紐付けを行うこと
  • 可能ならば priority ラベルを付けること
  • 可能ならば Assignees を設定すること
  • close するときは結論を明記すること
@200km
Copy link
Member

200km commented May 29, 2023

一応確認ですが、どのコンパイラでのwarningを想定していますか?
もちろん理想的には全てのコンパイラでwarningでないことが重要ですが、AOBCとしては現時点の最優先はvMicroコンパイラで、それだとc2a-core側のwarning対応などもでてくるかと思っています。

@suzuki-toshihir0
Copy link
Member Author

コンパイラ情報が抜けていましたが,vMicroでなく Clang on Linux32です.なので直近急いで対応が必要という感じではないと思ってます.

@200km
Copy link
Member

200km commented May 31, 2023

わかりました。では、priorityもlowに変えておきますね。

@200km 200km added 🚀 priority::high priority high 🐟 patch Patch Update labels Jul 14, 2023
@200km 200km added this to the 開発仮目標1 milestone Jul 14, 2023
@200km
Copy link
Member

200km commented Jul 14, 2023

#98

こちらである程度warningが潰されているので、要確認。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚗 priority::low priority low 🐟 patch Patch Update
Projects
None yet
Development

No branches or pull requests

2 participants