On 08/06/13 19:14, Ian Lynagh wrote:
commit 1f81187e63dcd4f160cfd1ce62efc4b1d26603dc
Author: Ian Lynagh <[email protected]>
Date:   Sat Jun 8 18:54:23 2013 +0100

     Remove uses of RecordWildCards in GHC.Event.TimerManager

curious: why?

@@ -174,10 +173,10 @@ finished :: TimerManager -> IO Bool
  finished mgr = (== Finished) `liftM` readIORef (emState mgr)

  cleanup :: TimerManager -> IO ()
-cleanup TimerManager{..} = do
-  writeIORef emState Finished
-  I.delete emBackend
-  closeControl emControl
+cleanup mgr = do
+  writeIORef (emState mgr) Finished
+  I.delete (emBackend mgr)
+  closeControl (emControl mgr)

Note that in general this transformation might lead to worse code if the field selectors aren't used strictly. (I don't think it's the case here, but I didn't check the rest of the diff.)

Cheers,
        Simon


_______________________________________________
ghc-devs mailing list
[email protected]
http://www.haskell.org/mailman/listinfo/ghc-devs

Reply via email to