googleapis / googleapis/google-cloud-cpp
Make sure we use `google::protobuf::compiler::cpp::FieldName` where needed when emitting code
オープン
cpp: generator
type: cleanup
- 主要言語
- C++
- スター
- 659
- フォーク
- 462
- 平均マージ
- 1日 2時間
- マージ済み PR(30日)
- 89
説明
In `generator/internal/http_option_utils.cc`, and maybe some other places, we emit code that uses the field name directly without first running it through `google::protobuf::compiler::cpp::FieldName` which handles the case where a field name is a C++ keyword or other disallowed token.
コントリビューションガイド
調査の方向性
まず generator/internal/http_option_utils.cc を読み、google::protobuf::compiler::cpp::FieldName を経由せずに出力される各フィールド名を特定します。近くの generator コードで同様にフィールド名を直接出力している箇所を検索し、その後、影響を受けるすべての生成識別子が C++ のキーワードやその他の使用できないトークンを正しく処理していることを確認します。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- cpp
- 領域
- compilers
- issue の種類
- バグ
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 45/100