Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP: Add Playfair cracker #13

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

RotationMatrix
Copy link

Companion to my PR in Ciphey.

@todo
Copy link

todo bot commented Oct 24, 2020

Ciphertext is not playfair if is contains a repeated digraph (AA, BB, CC, etc.)

// TODO: Ciphertext is not playfair if is contains a repeated digraph (AA, BB, CC, etc.)
// TODO: Ciphertext is not playfair if is contains more than 25 distinct letters.
// if (observed.size() > 25) {
// return 0.;
// }


This comment was generated by todo based on a TODO comment in ca59031 in #13. cc @RotationMatrix.

@todo
Copy link

todo bot commented Oct 24, 2020

Ciphertext is not playfair if is contains more than 25 distinct letters.

// TODO: Ciphertext is not playfair if is contains more than 25 distinct letters.
// if (observed.size() > 25) {
// return 0.;
// }
// For now, we can ignore ciphertext with non-Latin characters.


This comment was generated by todo based on a TODO comment in ca59031 in #13. cc @RotationMatrix.

@todo
Copy link

todo bot commented Oct 24, 2020

Use ciphertext to guess the 26th letter.

// TODO: Use ciphertext to guess the 26th letter.
std::vector<char_t> alphabet {
'a', 'b', 'c', 'd', 'e',
'f', 'g', 'h', 'i', 'k',
'l', 'm', 'n', 'o', 'p',
'q', 'r', 's', 't', 'u',


This comment was generated by todo based on a TODO comment in ca59031 in #13. cc @RotationMatrix.

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.

None yet

1 participant