[ https://issues.apache.org/jira/browse/THRIFT-3027?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14351563#comment-14351563 ]
Jens Geyer commented on THRIFT-3027: ------------------------------------ Hi [~magrath], the patch introduces a new dependency to boost into the Thrift compiler. Can we avoid that? Have a look into {{t_c_glib_generator.cc}}, there are some similar functions that may serve as model. Next, the {{to_upper_copy()}} function expects a locale, the default is {{std::locale()}} which is ... {quote} Default constructor. Constructs a copy of the global C++ locale, which is *the locale most recently used as the argument to std::locale::global or a copy of std::locale::classic if no call to std::locale::global has been made*. {quote} Not sure if that is what we want. The generated code should not depend on a particular system locale. > Go compiler does not ensure common initialisms have consistent case > ------------------------------------------------------------------- > > Key: THRIFT-3027 > URL: https://issues.apache.org/jira/browse/THRIFT-3027 > Project: Thrift > Issue Type: Improvement > Components: Go - Compiler > Affects Versions: 0.9.2 > Reporter: Paul Magrath > Assignee: Jens Geyer > Priority: Minor > > In Go, as per > https://github.com/golang/go/wiki/CodeReviewComments#initialisms, words in > names that are initialisms or acronyms should have a consistent case. > For example, if you have a struct like: > {code} > struct InitialismsTest { > 1: string user_id, > 2: string server_url, > } > {code} > One would expect it to compile to: > {code} > type InitialismsTest struct { > UserID string `thrift:"user_id,1" json:"user_id"` > ServerURL string `thrift:"server_url,2" json:"server_url"` > } > {code} > Rather than:- > {code} > type InitialismsTest struct { > UserId string `thrift:"user_id,1" json:"user_id"` > ServerUrl string `thrift:"server_url,2" json:"server_url"` > } > {code} > It would be pretty difficult to handle all cases of initialisms in the Go > compiler of course, but there is a set of common initialisms that have been > identified by the authors of Golint and could be handled relatively easily:- > https://github.com/golang/lint/blob/master/lint.go#L692 -- This message was sent by Atlassian JIRA (v6.3.4#6332)