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

Michael Greene commented on THRIFT-204:
---------------------------------------

I don't have any immediate objects, but I haven't tested this out yet.
I'd actually be in favor of making all classes  {{sealed}} and {{partial}}, but 
I haven't looked deep into the ramifications of this either.  Thoughts?

I'll take a look at some of your other issues as well.

> C# Partial Classes
> ------------------
>
>                 Key: THRIFT-204
>                 URL: https://issues.apache.org/jira/browse/THRIFT-204
>             Project: Thrift
>          Issue Type: Improvement
>          Components: Compiler (C#)
>         Environment: all
>            Reporter: Anthony Abate
>            Priority: Critical
>         Attachments: partial classes.patch
>
>   Original Estimate: 0.17h
>  Remaining Estimate: 0.17h
>
> Generated classes should be "partial"
> most code generated from Microsoft code generation tools these days is 
> "partial".  This way a developer can easily extend the class.
> If i found the correct line in the t_csharp_generator.cc , then his is a very 
> easy fix.  

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to