googleapis / googleapis/google-cloud-cpp

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

Đang mở
#14,505 0 bình luận 0 reaction 0 người được giao Xem trên GitHub
cpp: generator type: cleanup
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

Mở 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

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.