You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "Roger Meier (JIRA)" <ji...@apache.org> on 2014/11/16 22:48:34 UTC

[jira] [Reopened] (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:all-tabpanel ]

Roger Meier reopened THRIFT-2647:
---------------------------------

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