[ https://issues.apache.org/jira/browse/THRIFT-2689?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14162400#comment-14162400 ]
Hudson commented on THRIFT-2689: -------------------------------- SUCCESS: Integrated in Thrift #1291 (See [https://builds.apache.org/job/Thrift/1291/]) THRIFT-2689 struct names that differ only in capitalization of first character generate broken erlang code (jensg: rev dd971d5377c3e0271ffb9692217196274a8e02e4) * test/NameConflictTest.thrift * test/erl/src/name_conflict_test.erl > struct names that differ only in capitalization of first character generate > broken erlang code > ---------------------------------------------------------------------------------------------- > > Key: THRIFT-2689 > URL: https://issues.apache.org/jira/browse/THRIFT-2689 > Project: Thrift > Issue Type: Bug > Components: Erlang - Compiler, Erlang - Library > Reporter: alisdair sullivan > Assignee: alisdair sullivan > Priority: Minor > Fix For: 0.9.2 > > > the following thrift specification generates broken erlang code: > struct invalid { /* contents of struct are irrelevent */ } > struct Invalid { /* still irrelevent } > the compiler currently renames any struct that starts with a capital so that > it starts with a lowercase letter. instead, it should leave names intact and > use proper erlang syntax in generation. the attached patch does exactly that > this does break backwards compatibility -- This message was sent by Atlassian JIRA (v6.3.4#6332)