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

David Reiss commented on THRIFT-42:
-----------------------------------

I think that in this particular case, it would be best as an option to the java 
generator.  --gen java:beans,nocamel

It would not be too hard to add a generic directive to the thrift files like:
setoption "key" "value"
But doing it per-structure or per-field  is trickier.  Of course, there are 
already docstrings on those, so we could start inspecting them (HACK!).

> javabean generated code assumes camel-case and makes very ugly getters and 
> setters if not camel cased properly
> --------------------------------------------------------------------------------------------------------------
>
>                 Key: THRIFT-42
>                 URL: https://issues.apache.org/jira/browse/THRIFT-42
>             Project: Thrift
>          Issue Type: Improvement
>          Components: Compiler (Java)
>            Reporter: Bryan Duxbury
>            Priority: Trivial
>
> Let's say you name your fields in underscored lower case (my_cool_field). The 
> bean-style generator only upcases the first character of the name and mashes 
> it onto get, set, etc. You end up with something that looks like 
> "getMy_cool_field". This gross hybrid is pretty terrible. 
> I propose that we either add a case-style compiler option, or at least try 
> and detect the casing and convert it appropriately.

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