-fth-dec-file can't handle lambdas

instance Data.TH.Object.Object InputDropboxFiles String Language.JavaScript.Interpret.Primitive where
  toObject x@(InputDropboxFiles {})
    = fromList
        (Data.Maybe.catMaybes
           [Just
              (\ k_axsq v_axsr -> (k_axsq, Data.TH.Convert.convert v_axsr)
                 "root" (inputDropboxFilesRoot x)),
            Just
              (\ k_axsq v_axsr -> (k_axsq, Data.TH.Convert.convert v_axsr)
                 "path" (inputDropboxFilesPath x)),
            fmap
              (\ k_axsq v_axsr -> (k_axsq, Data.TH.Convert.convert v_axsr) "rev")
              (inputDropboxFilesRev x),
            Just
              (\ k_axsq v_axsr -> (k_axsq, Data.TH.Convert.convert v_axsr)
                 "access_token" (inputDropboxFilesAccess_token x))])

(\ k_axsq v_axsr -> (k_axsq, Data.TH.Convert.convert v_axsr)
                 "root" (inputDropboxFilesRoot x))

should be

((\ k_axsq v_axsr -> (k_axsq, Data.TH.Convert.convert v_axsr))
                 "root" (inputDropboxFilesRoot x))

ie. the lambda needs parentheses around it.

mentioned in issue #10603 (closed)

changed weight to 5

added Tbug Trac import labels

changed the description

Edited to what I think you meant.

Replying to [ticket:10703#comment:102976 rwbarton]:

Edited to what I think you meant.

Yes thanks

Trac metadata

Trac field	Value
Component	Compiler → Template Haskell

added TemplateHaskell label

Fabian: if you change your test to not depend on any external libraries, that would be great. The problem is we can't put it in the testsuite as is.

cc GregWeber, who implemented this feature recently.

This probably indicates a general problem with TH pretty-printing. Is the problem the same with -ddump-splices ?

Replying to [ticket:10703#comment:103110 GregWeber]:

This probably indicates a general problem with TH pretty-printing. Is the problem the same with -ddump-splices ?

Yes, I have updated the test with output from both -dth-dec-file and -ddump-splices

Attached file Test10702.hs ($1118).

Replying to [ticket:10703#comment:103100 thomie]:

Fabian: if you change your test to not depend on any external libraries, that would be great. The problem is we can't put it in the testsuite as is.

cc GregWeber, who implemented this feature recently.

Done

Great. To summarize, the output of ghc -ddump-splices Test10702.hs is:

instance Read Name where
     readsPrec x = \ k_a46F v_a46G -> undefined "test" "test2"

And it should be:

instance Read Name where
     readsPrec x = (\ k_a46F v_a46G -> undefined) "test" "test2"

This is being worked on in #10603 (closed).

Trac metadata

Trac field	Value
Related	#10701 → #10603 (closed), #10701

changed milestone to %8.0.1

Milestone renamed

closed

This should be fixed along with #10603 (closed).

Trac metadata

Trac field	Value
Resolution	Unresolved → ResolvedFixed

added Pnormal label