#4148: improve new recursive do syntax
---------------------------------+------------------------------------------
Reporter: guest | Owner:
Type: feature request | Status: patch
Priority: low | Milestone: 7.6.1
Component: Compiler | Version: 6.12.3
Keywords: | Os: Unknown/Multiple
Architecture: Unknown/Multiple | Failure: None/Unknown
Difficulty: Unknown | Testcase:
Blockedby: | Blocking:
Related: |
---------------------------------+------------------------------------------
Changes (by lerkok):
* status: new => patch
Comment:
Simon: It turned out that the required changes for option B were rather
simple. (I have
not done anything to reject `rec` blocks inside `mdo`, the code for that
looked unnecessarily
complicated without much benefit.)
I've run the "validate" script, I had to make one change to get it going:
* Commented out line 244 of libraries/base/System.IO.hs; "validate" was
complaining
that the `import Foreign.C.String` on that line was not used. (Looks
like "validate"
is using -Werror for certain calls?)
Since this had nothing to do with my changes, I'm assuming there's a
problem with "validate"
in the main branch as it exists today. Once I commented that line,
"validate" finished, failing
only three test cases:
* numeric/should_run numrun012 [bad stdout] (normal)
* perf/compiler T1969 [stat not good enough] (normal)
* perf/compiler T3064 [stat not good enough] (normal)
I think these failures are not relevant to the changes I'm making. The
first one seems to be the
only one that's rather troubling. Let me know if you'd like me to pursue
it any further.
The latter two are "stat not good enough" kind, which is to be
expected on my machine. (I'm running on Ubuntu on top of a VirtualBox VM.)
I've created two patches:
* mdo_masterPatch.diff : Changes to code + documentation; please apply
in main repo
* mdo_testSuitePatch.diff: Changes to the test-suite; please apply in
the testSuite repo
I don't see a way to attach the patches to this ticket; so I'll mail them
to you separately. While the changes were rather trivial, please do take a
look and let me know if you spot anything fishy.
Thanks,
-Levent.
--
Ticket URL: <http://hackage.haskell.org/trac/ghc/ticket/4148#comment:25>
GHC <http://www.haskell.org/ghc/>
The Glasgow Haskell Compiler
_______________________________________________
Glasgow-haskell-bugs mailing list
[email protected]
http://www.haskell.org/mailman/listinfo/glasgow-haskell-bugs
