[
https://issues.apache.org/jira/browse/THRIFT-897?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12908188#action_12908188
]
Bryan Duxbury commented on THRIFT-897:
--------------------------------------
I think the smart resolution thing is only worthwhile if we really want to keep
backwards compatibility. I'm not crazy about breaking people's IDL, but I think
that's preferable in the long term to supporting a more convoluted syntax.
If we were starting from scratch today, I think that we'd want to use the
fully-qualified approach, and the cost of fixing the breakage *should* be
reasonably low, especially because you're getting a consistency improvement at
the same time. This is certainly my preference.
> Don't allow unqualified constant access to enum values
> ------------------------------------------------------
>
> Key: THRIFT-897
> URL: https://issues.apache.org/jira/browse/THRIFT-897
> Project: Thrift
> Issue Type: Bug
> Components: Compiler (General)
> Affects Versions: 0.1, 0.2, 0.3, 0.4
> Reporter: Bryan Duxbury
> Fix For: 0.5
>
>
> Through looking at THRIFT-544 and THRIFT-895, it's come to my attention that
> we currently register each of every enum's values as a global (and scoped)
> constant. This allows you to do things like:
> {code}
> enum MyEnum {
> A = 1
> B = 2
> }
> const MyEnum myEnumVar = A;
> {code}
> This is handy, insofar as you might want to use the values of an enum in
> constant or default circumstances. However, this behavior is unstable - if
> you have two enums with values that have the same name, all constant
> references will point at the last occurrence of the name. Further, in order
> to allow this to go on, we must not check if any constant has been declared
> twice, which means you can get stupid, detectable errors in your IDL very
> easily.
> I propose that we stop allowing this method of access, and instead require
> the enum values referenced in constant context to be prefixed with the enum
> type's name. For instance:
> {code}
> enum MyEnum {
> A = 1
> B = 2
> }
> const MyEnum myEnumVar = MyEnum.A;
> {code}
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
