[
https://issues.apache.org/jira/browse/THRIFT-1318?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13598366#comment-13598366
]
Hudson commented on THRIFT-1318:
--------------------------------
Integrated in Thrift #629 (See [https://builds.apache.org/job/Thrift/629/])
THRIFT-1318 Incorrect syntax for struct with enum property and default
value when value is negative (Revision b3fb3e60925089b6d4c672d7d2cff76a94303b6a)
Result = FAILURE
jensg :
https://git-wip-us.apache.org/repos/asf?p=thrift.git&a=commit&h=b3fb3e60925089b6d4c672d7d2cff76a94303b6a
Files :
* compiler/cpp/src/generate/t_csharp_generator.cc
> Incorrect syntax for struct with enum property and default value when value
> is negative
> ---------------------------------------------------------------------------------------
>
> Key: THRIFT-1318
> URL: https://issues.apache.org/jira/browse/THRIFT-1318
> Project: Thrift
> Issue Type: Bug
> Components: C# - Compiler
> Affects Versions: 0.7
> Environment: Windows 7 / .net 4.0
> Reporter: Thunder Stumpges
> Assignee: Jens Geyer
> Attachments: NegativeEnumConstant.patch,
> THRIFT-1318_Incorrect_syntax_for_struct_with_enum_property_and_default_value_when_value_is_negative_rev2.patch
>
> Original Estimate: 1h
> Remaining Estimate: 1h
>
> If I have a struct with a property of type enum with a default value
> specified, and the default value is negative, then the code generated is
> incorrect and has a syntax error.
> E.G. thrift file like this:
> {code}
> enum ReturnType {
> SuccessTypeA = 2
> SuccessUnknownType = 1,
> FailUnknown = -1,
> OtherFailure = -2
> }
> struct ReturnInformation {
> 1: required ReturnType RetType = ReturnType.FailUnknown;
> }
> {code}
> Generates a constructor like this:
> {code}
> public MatchInformation() {
> this._MatchType = (MatchTypes)-1;
> }
> {code}
> which fails with an exception "To cast a negative value, you must enclose the
> value in parentheses".
> I have updated the code to always put the enum constant value in parentheses
> (positive or negative), but need to get the compiler built and tested before
> I can submit a patch. Any chance someone could do this for me?
> :)
>
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
