|
Message-ID: <BLU0-SMTP21B7BA5908C59B59B78A86FDE10@phx.gbl>
Date: Mon, 25 Jun 2012 21:44:32 +0200
From: Frank Dittrich <frank_dittrich@...mail.com>
To: john-dev@...ts.openwall.com
Subject: Markov: move mkv_start / mkv_end computation out of do_markov_crack()
(was: committed patches)
On 06/25/2012 08:46 PM, magnum wrote:
> Committed now. Meld is a frigging excellent tool, I did not even get the
> merging problems I feared in the contest tree.
Because you are a brave man, I have a new patch for you.
This one doesn't introduce any new logic, I was just refactoring
existing code.
Now the mkv_start and mkv_end computation is moved out of
do_markov_crack() and into a new function get_markov_start_end().
Just the MPI specific mkv_start / mkv_end adjustment remains in
do_markov_crack().
I compiled each intermediate transformation step, and verified that
these commands all behaved like they did before my refactoring:
$ ./john --stdout --markov:200:1%:2%:12 |head -n 1
$ ./john --stdout --markov:200:1%:2%:12 |tail -n 1
$ ./john --stdout --markov:200:2593564:5187128:12 |head -n 1
$ ./john --stdout --markov:200:2593564:5187128:12 |tail -n 1
> I did not drop the docs, we can just as well have it documented. I bet
> more people will use this than MPI anyway (doh!).
May be it is better documented or easier to use than MPI is?
Frank
View attachment "0001-Markov-move-mkv_start-mkv_end-computation-out-of-do_.patch" of type "text/x-patch" (6254 bytes)
Powered by blists - more mailing lists
Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.