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

Cmd DB から "Cmd_" prefix を削除 #20

Merged
merged 1 commit into from
Apr 9, 2023
Merged

Conversation

meltingrabbit
Copy link
Collaborator

@meltingrabbit meltingrabbit commented Apr 3, 2023

概要

Cmd DB から "Cmd_" prefix を削除

Issue

詳細

以下とともにマージする

検証結果

影響範囲

  • 軽微な修正だが,他の Tools も変わるので, v2.4.0 を打つ

@chutaro
Copy link

chutaro commented Apr 6, 2023

一応確認ですが、Cmd DB のエクセル自体には変更必要なくて、どんな prefix でも問題ないですよね?(あくまでテレコマgeneratorでどんな変換をするか、ですよね?)

@meltingrabbit
Copy link
Collaborator Author

一応確認ですが、Cmd DB のエクセル自体には変更必要なくて、どんな prefix でも問題ないですよね?(あくまでテレコマgeneratorでどんな変換をするか、ですよね?)

というと?

ut-issl/c2a-tlm-cmd-code-generator#35

のとおり,

  • 完全user定義(generator で Cmd_ つけて,C の namespace 確保)
  • DBで Cmd_ までつけちゃう(広報互換性確保)

のどちらかだよ

@chutaro

@chutaro
Copy link

chutaro commented Apr 6, 2023

はい、すみませんそこは理解しています。

Cmd にどんな prefix を付けるか、どんな置換処理をするかは tlm cmd generator の実装しだいで、Cmd DB はコマンド名にどんな値が入っていたとしてもそのまま csv に出すだけですよね、という意味で確認したかった感じです

@chutaro
Copy link

chutaro commented Apr 6, 2023

approve してます

@meltingrabbit meltingrabbit merged commit 3e86c97 into main Apr 9, 2023
@meltingrabbit meltingrabbit deleted the feature/del_cmd_prefix branch April 9, 2023 13:56
@meltingrabbit
Copy link
Collaborator Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority::high priorityg high
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants