RE: [commit: ghc] wip/T10613: State hack hack in ticky report (02493dc)

Joachim
The commit message says something about "tick reports", but the
change to the code is entirely un-commented, and appears to have
nothing to do with ticky reports.
Could you add a comment or Note to explain?
Thanks!
Simon
| -----Original Message-----
| From: ghc-commits [mailto:ghc-commits-bounces@haskell.org] On Behalf Of
| git@git.haskell.org
| Sent: 14 April 2016 21:35
| To: ghc-commits@haskell.org
| Subject: [commit: ghc] wip/T10613: State hack hack in ticky report
| (02493dc)
|
| Repository : ssh://git@git.haskell.org/ghc
|
| On branch : wip/T10613
| Link :
| https://na01.safelinks.protection.outlook.com/?url=http%3a%2f%2fghc.has
| kell.org%2ftrac%2fghc%2fchangeset%2f02493dcf580141110e90fa943ec70640115
| de0ed%2fghc&data=01%7c01%7csimonpj%40064d.mgd.microsoft.com%7c306055dca
| afa407e7ce708d364a49c11%7c72f988bf86f141af91ab2d7cd011db47%7c1&sdata=d%
| 2f7isihh6dMDZX93hTY8rsMQ7y%2f6mkiDm2asfPXpMrQ%3d
|
| >---------------------------------------------------------------
|
| commit 02493dcf580141110e90fa943ec70640115de0ed
| Author: Joachim Breitner

Dear Simon, Am Freitag, den 15.04.2016, 11:04 +0000 schrieb Simon Peyton Jones:
The commit message says something about "tick reports", but the change to the code is entirely un-commented, and appears to have nothing to do with ticky reports.
Could you add a comment or Note to explain?
sorry that this happens so often; but this is again just a work-in- progress-commit on a wip branch, and as such not yet in the polished stat required to go to master. This commit makes a bit more sense when seen in conjunction with 30b9061317ea7b834339d9458c46bd6b9108d947 Be more explicit about closure types in ticky-ticky-report which is already in master (and where, upon reading right now, I’m not sure if the current code mkLFReEntrant top fvs args arg_descr = LFReEntrant top os_info (length args) (null fvs) arg_descr where os_info = idOneShotInfo (head args) is really the right away, or whether the idOneShotInfo of all arguments should be used here (I think you said something along these lines, I failed to follow up on it). Greetings, Joachim -- Joachim “nomeata” Breitner mail@joachim-breitner.de • https://www.joachim-breitner.de/ XMPP: nomeata@joachim-breitner.de • OpenPGP-Key: 0xF0FBF51F Debian Developer: nomeata@debian.org
participants (2)
-
Joachim Breitner
-
Simon Peyton Jones