チケット #48786

specenum generator: Value prefix in header

登録: 2023-10-02 05:51 最終更新: 2023-10-05 18:14

報告者:
担当者:
チケットの種類:
状況:
完了
コンポーネント:
マイルストーン:
優先度:
5 - 中
重要度:
5 - 中
解決法:
修正済み
ファイル:
1

詳細

Currently each value in specenum generator .def file contains the prefix, e.g., MANUAL_SETTINGS, MANUAL_COMMANDS ...

Should we move the prefix to header part, and define values simply as SETTINGS, COMMANDS ... ?

チケットの履歴 (8 件中 3 件表示)

2023-10-02 05:51 更新者: cazfi
  • 新しいチケット "specenum generator: Valure prefix in header" が作成されました
2023-10-02 05:56 更新者: cazfi
コメント

Having support for prefix in header would not prevent one from leaving prefix empty, and still writing all of it to individual values.

2023-10-02 06:02 更新者: cazfi
  • 概要が更新されました
2023-10-02 16:45 更新者: alienvalkyrie
コメント

This was something I'd been thinking of from the start (but left out for the sake of minimal initial implementation). The only reservation I have about this is that when searching source files (e.g. git grep or just searching on github) for one of the enum constants, right now you'd still find the corresponding line in the .def at least; if we change this, then once the generated headers are no longer checked into git, you wouldn't be able to find the definition at all if you don't already know where to look.

Then again, the same is already true for all the auxiliary functions; and you would still be able to find the name of the enum type itself.

2023-10-02 18:07 更新者: alienvalkyrie
  • 担当者(未割り当て) から alienvalkyrie に更新されました
  • 解決法なし から 受領 に更新されました
コメント

Attached patch. Using the prefix is optional, so it's backwards compatible with other enums converted to the new format in the interim (e.g. #48785).

I don't like that this now leads to writing count COUNT a bunch; will make a follow-up for that.

2023-10-02 18:37 更新者: alienvalkyrie
コメント

Reply To alienvalkyrie

I don't like that this now leads to writing count COUNT a bunch; will make a follow-up for that.

~> #48789

2023-10-05 18:14 更新者: alienvalkyrie
  • 状況オープン から 完了 に更新されました
  • 解決法受領 から 修正済み に更新されました

編集

このチケットにコメントを追加するには、ログインが必要です » ログインする