You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "Bryan Duxbury (JIRA)" <ji...@apache.org> on 2010/09/20 17:32:35 UTC
[jira] Commented: (THRIFT-906) Improve type mappings
[ https://issues.apache.org/jira/browse/THRIFT-906?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12912500#action_12912500 ]
Bryan Duxbury commented on THRIFT-906:
--------------------------------------
For users updating to a newer version of Thrift, what kind of changes will the necessitate?
> Improve type mappings
> ---------------------
>
> Key: THRIFT-906
> URL: https://issues.apache.org/jira/browse/THRIFT-906
> Project: Thrift
> Issue Type: New Feature
> Components: Haskell - Compiler, Haskell - Library
> Environment: Darwin -- 10.4.0 Darwin Kernel Version 10.4.0: Fri Apr 23 18:28:53 PDT 2010; root:xnu-1504.7.4~1/RELEASE_I386 i386
> The Glorious Glasgow Haskell Compilation System, version 6.12.3
> Reporter: Christian Lavoie
> Fix For: 0.5, 0.6, 0.7
>
> Attachments: thrift-improved-haskell-type-mappings.patch
>
>
> The current haskell type mappings are awkward and error prone to work with:
> binary -> String
> string -> String
> byte -> Int
> i16 -> Int
> i32 -> Int
> i64 -> Int
> This patch updates the mappings to the canonical types of the correct length in Haskell:
> binary -> Data.ByteString.Lazy.ByteString
> string -> String
> byte -> Data.Word.Word8
> i16 -> Data.Int.Int16
> i32 -> Data.Int.Int32
> i64 -> Data.Int.Int64
> THIS BREAKS EXISTING CODE. It is, however, very arguably broken already.
> For convenience of patching, this patch is a superset of THRIFT-743.
> Thoughts?
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.