Skip to content
This repository has been archived by the owner on Jul 2, 2022. It is now read-only.

Removed unnecessary fulltest #45

Merged
merged 1 commit into from Apr 3, 2013
Merged

Removed unnecessary fulltest #45

merged 1 commit into from Apr 3, 2013

Conversation

toomyem
Copy link
Contributor

@toomyem toomyem commented Apr 3, 2013

I think this is unnecessary call to function fulltest() for cryptopp_asm32 method.

@jgarzik
Copy link
Owner

jgarzik commented Apr 3, 2013

Why do you feel it is unnecessary?

@toomyem
Copy link
Contributor Author

toomyem commented Apr 3, 2013

Because it is computed during the if evaluation. Why call it again once
more?
3 kwi 2013 08:11, "Jeff Garzik" notifications@github.com napisał(a):

Why do you feel it is unnecessary?

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/45#issuecomment-15820131
.

@toomyem toomyem closed this Apr 3, 2013
@toomyem toomyem reopened this Apr 3, 2013
jgarzik pushed a commit that referenced this pull request Apr 3, 2013
Merge pull request #45 from toomyem/master
0d54567 
Removed unnecessary fulltest
@jgarzik jgarzik merged commit 0d54567 into jgarzik:master Apr 3, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@toomyem @jgarzik