Skip to content

Commit

Permalink
Fix comment
Browse files Browse the repository at this point in the history
Summary: Causes user confusion by pointing to deprecated unstructured annotation

Reviewed By: Mizuchi

Differential Revision: D51990713

fbshipit-source-id: 735975d4613047d6a65d18e322ee1894848e417d
  • Loading branch information
iahs authored and facebook-github-bot committed Dec 9, 2023
1 parent ffda496 commit 8667fbc
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion thrift/annotation/cpp.thrift
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@ enum RefType {

/**
* Changes the name of the definition in generated C++ code.
* In most cases a much better solution is to rename the problematic Thrift field itself. Only use the `cpp.name` annotation if such renaming is problematic,
* In most cases a much better solution is to rename the problematic Thrift field itself. Only use the `@cpp.Name` annotation if such renaming is problematic,
* e.g. when the field name appears in code as a string, particularly when using JSON serialization, and it is hard to change all usage sites.
*/
@scope.Definition
Expand Down

0 comments on commit 8667fbc

Please sign in to comment.