Cleanup of RnExpr, RnEnv and RnSource

Dear all, RnExpr, RnEnv and RnSource (in compiler/renamer) declare locally: \begin{code} -- XXX thenM :: Monad a => a b -> (b -> a c) -> a c thenM = (>>=) thenM_ :: Monad a => a b -> a c -> a c thenM_ = (>>) \end{code} and then use these aliases instead of >>= and >>. These were introduced by Ian in 6c7b41cc2b24f533697a62bf1843507ae043fc97 in 2008 and later partially cleaned up by Simon in f0c99958649b8909612b1b9c9b48aad970dfce05 in 2009. Would there be objections if I cleand this up (ie. remove thenM and thenM_ and use >>= and >>)? Is anyone doing any substantial changes on any of these files and suspects that this could introduce merge conflicts for him? I'm at the moment changing some things in RnExpr, so doing this will only add little extra work for me and I'd like to take this oportunity to have cleaner code. Janek

Or better still use do-notation like the rest of the compiler. That'd be great thatnks. You could de-tabify and remove white space at the same time SImon | -----Original Message----- | From: ghc-devs [mailto:ghc-devs-bounces@haskell.org] On Behalf Of Jan | Stolarek | Sent: 30 June 2014 10:28 | To: ghc-devs@haskell.org | Subject: Cleanup of RnExpr, RnEnv and RnSource | | Dear all, | | RnExpr, RnEnv and RnSource (in compiler/renamer) declare locally: | | \begin{code} | -- XXX | thenM :: Monad a => a b -> (b -> a c) -> a c thenM = (>>=) | | thenM_ :: Monad a => a b -> a c -> a c | thenM_ = (>>) | \end{code} | | and then use these aliases instead of >>= and >>. These were introduced | by Ian in | 6c7b41cc2b24f533697a62bf1843507ae043fc97 in 2008 and later partially | cleaned up by Simon in | f0c99958649b8909612b1b9c9b48aad970dfce05 in 2009. Would there be | objections if I cleand this up (ie. remove thenM and thenM_ and use >>= | and >>)? Is anyone doing any substantial changes on any of these files | and suspects that this could introduce merge conflicts for him? I'm at | the moment changing some things in RnExpr, so doing this will only add | little extra work for me and I'd like to take this oportunity to have | cleaner code. | | Janek | _______________________________________________ | ghc-devs mailing list | ghc-devs@haskell.org | http://www.haskell.org/mailman/listinfo/ghc-devs
participants (2)
-
Jan Stolarek
-
Simon Peyton Jones