googleapis / googleapis/google-cloud-cpp
Make sure we use `google::protobuf::compiler::cpp::FieldName` where needed when emitting code
- Ngôn ngữ chính
- C++
- Star
- 659
- Fork
- 462
- Merge trung bình
- 1 ngày 2 giờ
- Pull request đã merge (30 ngày)
- 89
Mô tả
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.
Hướng dẫn đóng góp
Hướng nghiên cứu
Bắt đầu bằng cách đọc generator/internal/http_option_utils.cc và xác định từng tên trường được phát ra mà bỏ qua google::protobuf::compiler::cpp::FieldName. Tìm kiếm trong phần mã generator lân cận các trường hợp phát ra tên trường trực tiếp tương tự, sau đó xác minh rằng tất cả các mã định danh được tạo bị ảnh hưởng đều xử lý đúng các từ khóa C++ và những token không được phép khác.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- cpp
- Lĩnh vực
- compilers
- Loại issue
- Lỗi
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 45/100