You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "Hudson (JIRA)" <ji...@apache.org> on 2013/09/06 23:03:51 UTC

[jira] [Commented] (THRIFT-2166) Delphi compiler should generate constant class name prefixed with thrift file name (similar to Java and C#)

    [ https://issues.apache.org/jira/browse/THRIFT-2166?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13760621#comment-13760621 ] 

Hudson commented on THRIFT-2166:
--------------------------------

SUCCESS: Integrated in Thrift #888 (See [https://builds.apache.org/job/Thrift/888/])
THRIFT-2166 Delphi compiler should generate constant class name prefixed with thrift file name (similar to Java and C#) (jensg: rev ec8744c4dd84f8077a590cdcbb58a921cec1d14c)
* compiler/cpp/src/generate/t_delphi_generator.cc

                
> Delphi compiler should generate constant class name prefixed with thrift file name (similar to Java and C#)
> -----------------------------------------------------------------------------------------------------------
>
>                 Key: THRIFT-2166
>                 URL: https://issues.apache.org/jira/browse/THRIFT-2166
>             Project: Thrift
>          Issue Type: Improvement
>          Components: Delphi - Compiler
>            Reporter: Jens Geyer
>            Assignee: Jens Geyer
>             Fix For: 1.0
>
>         Attachments: THRIFT-2166_Delphi__constant_class_name_prefixed_with_file_name.patch
>
>
> Similar to Java and C#, the Delphi compiler should generate constant class name prefixed with thrift file Name: 
> Example: {code}TIncludesTestConstants{code} instead of just {code}TConstants{code}.
> This makes it easier to distinguish between different constants when two or more generated Delphi units are used from the same source file without the need to add (possibly lengthy) prefixes. 

--
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