KarboniteKream commented on code in PR #2765:
URL: https://github.com/apache/thrift/pull/2765#discussion_r1161114230
##########
compiler/cpp/src/thrift/generate/t_oop_generator.h:
##########
@@ -70,7 +70,7 @@ class t_oop_generator : public t_generator {
}
virtual void generate_java_doc(std::ostream& out, t_field* field) {
- if (field->get_type()->is_enum()) {
+ if (get_true_type(field->get_type())->is_enum()) {
Review Comment:
Sure thing! This `@see` is already appended to some docs in files for
`TestDefinitionOrder` (it's how I found I needed to change this line), but I'll
add some more docs to `.thrift` files to confirm it's appended correctly.
What did you have in mind for asserting the result? I can't think of
anything else other than checking whether the generated file contains a `/* ...
@see ... */` string. 🤔
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
