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

Todd Lipcon commented on THRIFT-127:
------------------------------------

Two comments:

1) Do other languages support sloppiness between int types? It seems to me that 
if you change an i32 to an i64 for example, it should allow it (possibly with a 
warning) given erlang ints don't have sizes. If we're worried about overflow in 
dependent parts of the system, we could allow the sloppiness only in the upward 
direction (e.g. receiving an i32 when you expect an i64 isn't a problem).

2)  Line 150: FTypeAtom = thrift_protocol:typeid_to_atom(FType) is now 
redundant with Type.

> Erlang assumes that field types are correct and de-synchronizes if they are 
> not
> -------------------------------------------------------------------------------
>
>                 Key: THRIFT-127
>                 URL: https://issues.apache.org/jira/browse/THRIFT-127
>             Project: Thrift
>          Issue Type: Bug
>            Reporter: David Reiss
>            Assignee: David Reiss
>            Priority: Minor
>
> Patch in a moment.

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