googleapis / googleapis/google-cloud-cpp

Make sure we use `google::protobuf::compiler::cpp::FieldName` where needed when emitting code

オープン
#14,505 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
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

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。