You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "Hudson (JIRA)" <ji...@apache.org> on 2014/11/05 05:48:50 UTC
[jira] [Commented] (THRIFT-2647) compiler-hs: don't decapitalize
field names, do decapitalize argument bindings
[ https://issues.apache.org/jira/browse/THRIFT-2647?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14197518#comment-14197518 ]
Hudson commented on THRIFT-2647:
--------------------------------
SUCCESS: Integrated in Thrift #1331 (See [https://builds.apache.org/job/Thrift/1331/])
THRIFT-2647 compiler-hs: don't decapitalize field names, do decapitalize argument bindings (roger: rev 16c164ed3054b4edc65992a2b781307116fbca56)
* test/hs/NameConflictTest_Main.hs
* compiler/cpp/src/generate/t_hs_generator.cc
* test/NameConflictTest.thrift
* test/hs/Makefile.am
> compiler-hs: don't decapitalize field names, do decapitalize argument bindings
> ------------------------------------------------------------------------------
>
> Key: THRIFT-2647
> URL: https://issues.apache.org/jira/browse/THRIFT-2647
> Project: Thrift
> Issue Type: Bug
> Components: Haskell - Compiler
> Reporter: John Chee
> Assignee: Roger Meier
> Fix For: 0.9.2
>
> Attachments: 0001-compiler-hs-don-t-decapitalize-field-names-do-decapi.patch
>
>
> By modifying the haskell compiler slightly we can enable the NameConflictTest.
> {{make check}} in test/hs succeeds with:
> {code}
> sh run-test.sh NameConflictTest
> Running test...
> ../../lib/hs/src/Thrift/Types.hs:27:10: Warning:
> Orphan instance:
> instance (Hashable k, Hashable v) => Hashable (Map.HashMap k v)
> ../../lib/hs/src/Thrift/Types.hs:30:10: Warning:
> Orphan instance: instance Hashable a => Hashable (Set.HashSet a)
> ../../lib/hs/src/Thrift/Types.hs:33:10: Warning:
> Orphan instance: instance Hashable a => Hashable (Vector.Vector a)
> ../../test/hs/NameConflictTest_Main.hs:5:1: Warning:
> The import of ‘NameConflictTest_Consts’ is redundant
> except perhaps to import instances from ‘NameConflictTest_Consts’
> To import instances alone, use: import NameConflictTest_Consts()
> ../../test/hs/NameConflictTest_Main.hs:7:1: Warning:
> The import of ‘Qualified’ is redundant
> except perhaps to import instances from ‘Qualified’
> To import instances alone, use: import Qualified()
> ...
> Values:
> [JUST,TRUE,FALSE]
> [LEFT,RIGHT]
> Problem_ {f_Problem__problem = Just True, f_Problem__Problem = Just False}
> {code}
> Successful travis run: https://travis-ci.org/cheecheeo/thrift/jobs/31667026
> cc: [~roger.meier]
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)