-
Notifications
You must be signed in to change notification settings - Fork 93
add strict family function #49
Conversation
Core/Strict.hs
Outdated
) where | ||
|
||
($!) :: (a -> b) -> a -> b | ||
($!) f !a = f a |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok, my bad, this will conflict with Prelude's one. Can you remote it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah ok, did you mean "remove"? should I do a git revert on it?
On 24 July 2016 at 18:12, Nicolas DI PRIMA [email protected] wrote:
In Core/Strict.hs
#49 (comment)
:+-- Portability : portable
+--
+-- Enforce strictness when executing lambda
+--
+
+module Core.Strict
- ( ($!)
- , strict2
- , strict3
- , strict4
- , strict5
- , strict6
- ) where
+($!) :: (a -> b) -> a -> b
+($!) f !a = f aok, my bad, this will conflict with Prelude's one. Can you remote it?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/haskell-foundation/foundation/pull/49/files/56b57b3eb2b5bd9f3e9ca551fd1ead97ffe21013#r71991168,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACj1rvE8gheq6RgwyxN0jqBRYRDuUZLGks5qY47bgaJpZM4JTkbx
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no, juste delete the function and add the new commit.
The PR is not compiling on travis because the branch is missing some of @vincenthz 's last changes. Please, rebase your PR. |
Rename to indicate the absence of bounds checking. Add documenation specifying consequences if used improperly.
Add check for empty collections to avoid calling unsafe functions with an empty and potentially unsafe to use array.
In the plane, the brain connection to wikipedia knowledge was not available.
…noc-with-mempty Fix missing base case in cons/snoc for UTF8 strings
ok, this is not the way to do. I'll create a new PR with your commits. |
fix #16