Skip to content

c++ implementation#3

Open
lgrvs wants to merge 4 commits into
CrypTools:masterfrom
lgrvs:master
Open

c++ implementation#3
lgrvs wants to merge 4 commits into
CrypTools:masterfrom
lgrvs:master

Conversation

@lgrvs

@lgrvs lgrvs commented Apr 13, 2018

Copy link
Copy Markdown

No description provided.

lgrvs added 2 commits April 13, 2018 22:04
*Working* cpp implementation.
Requires gmp lib (<gmpxx.h>)
@CLAassistant

CLAassistant commented Apr 13, 2018

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@arguiot arguiot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good!

@arguiot arguiot requested a review from lukalafaye April 13, 2018 12:09

@arguiot arguiot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Having problem running this on other platforms. Requesting a patch

@arguiot

arguiot commented Apr 14, 2018

Copy link
Copy Markdown
Member

@el-famos0 Could you add a Makefile in your PR?

Make sure you include the following command:

clang++ -L /opt/local/lib -I /opt/local/include -lgmpxx -lgmp main.cpp

Added basic arguments support.
Multi-threading not implemented yet...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants