Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Suggested fix for Issue #105 #106

Merged
merged 5 commits into from Jan 14, 2013

Conversation

Projects
None yet
2 participants
Contributor

Aaronontheweb commented Jan 14, 2013

I decided to run an experiment and see if changing the CompositeType.GetHashCode function would alleviate the issue I described in issue #105

The unit test I attached to issue #105 now passes as a result of the changes made. I followed Jon Skeet's advice on generating safe hash functions for multiple objects in C# - what do you think?

P.S. sorry for the noise on this PR - had some trouble figuring out how to squash some commits from my previous PR.

Contributor

nberardi commented Jan 14, 2013

I like it. Thanks.

@nberardi nberardi added a commit that referenced this pull request Jan 14, 2013

@nberardi nberardi Merge pull request #106 from Aaronontheweb/issue105
Suggested fix for Issue #105
350f261

@nberardi nberardi merged commit 350f261 into fluentcassandra:master Jan 14, 2013

@ghost ghost assigned nberardi Jan 14, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment