Skip to content
This repository has been archived by the owner on Sep 3, 2024. It is now read-only.

Add more strictness to datatypes. #6

Closed
wants to merge 1 commit into from
Closed

Add more strictness to datatypes. #6

wants to merge 1 commit into from

Conversation

qnikst
Copy link
Contributor

@qnikst qnikst commented May 27, 2015

No description provided.

@facundominguez
Copy link
Contributor

It looks like using force here eliminates the need for this PR. Am I wrong?

@qnikst
Copy link
Contributor Author

qnikst commented May 27, 2015

I yes, we don't need this commit in order to solve the issue. However this change may be in spirit of the cloud-haskell where we have our basic construct strict with NFData instances.

1 similar comment
@qnikst
Copy link
Contributor Author

qnikst commented May 27, 2015

I yes, we don't need this commit in order to solve the issue. However this change may be in spirit of the cloud-haskell where we have our basic construct strict with NFData instances.

@qnikst
Copy link
Contributor Author

qnikst commented Jun 2, 2015

@facundominguez what to do with this PR, close as it's not required for the current use?

@qnikst
Copy link
Contributor Author

qnikst commented Jun 18, 2015

@facundominguez please take another look, this patch could be used instead of forcing in haskell-distributed/distributed-process#200

@@ -293,7 +296,7 @@ getStaticLabel :: Get StaticLabel
getStaticLabel = do
header <- getWord8
case header of
0 -> StaticLabel <$> get
0 -> mkStaticLabel <$> get
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No need to force here.

@qnikst
Copy link
Contributor Author

qnikst commented Jun 18, 2015

Fixed both

@facundominguez
Copy link
Contributor

I don't see any changes.

@qnikst
Copy link
Contributor Author

qnikst commented Jun 18, 2015

Is it better now?

@facundominguez
Copy link
Contributor

Merged as 953fbde.

@facundominguez facundominguez deleted the dev/more-strictness branch June 18, 2015 14:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants