[
https://issues.apache.org/jira/browse/THRIFT-2689?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
alisdair sullivan updated THRIFT-2689:
--------------------------------------
Description:
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
was:
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
> 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
> Priority: Minor
>
> 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)
