[
https://issues.apache.org/jira/browse/THRIFT-178?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Erik Frey updated THRIFT-178:
-----------------------------
Attachment: final_annotation_cpp_java_csharp_v2.patch
Okay, here's a final final annotation patch :-)
Includes suggestion to move the map.find to a separate line for clarity, and to
use "final" for consistency across languages (not to mention forward
compatibility with any other generators that may eventually support it).
Note that to use "final" I had to remove it from the lexer definition, where it
had been declared as a reserved word.
> Final Keyword
> -------------
>
> Key: THRIFT-178
> URL: https://issues.apache.org/jira/browse/THRIFT-178
> Project: Thrift
> Issue Type: New Feature
> Components: Compiler (C++), Compiler (General), Compiler (Java)
> Reporter: Erik Frey
> Priority: Minor
> Attachments: final_annotation_cpp_java_csharp.patch,
> final_annotation_cpp_java_csharp_v2.patch, final_csharp.patch,
> keyword_final_cpp_java.patch
>
>
> This introduces support for the final keyword in the thrift IDL. A C++
> thrift struct that is declared final loses it's virtual dtor, which saves the
> memory overhead of one vtable per instance. This (along with the required
> keyword) can be very helpful if you're passing around big lists of small
> thrift structs.
> This patch also includes modifications for the java generator to recognize
> and apply final. I'm not familiar enough with the other languages to know if
> this notion applies to them.
> (A patch for this was submitted in thrift's pre-apache days and was met with
> approval, but then fell off the radar, so I'm trying again.)
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
