Problems with prediction time. about scallion HOT 4 CLOSED

The problem looks to be this line in OnionToolConfig.cs. Line 15:
var hashes_per_win = _regex.GenerateAllPatternsForRegex().Select(t=>Math.Pow(2,5*t.Count(q=>q!='.') - 1)).Sum();

I don't think it should end in a Sum(). Having more possibilities should increase the chances of finding a solution, no? Also it is mentioned in the readme that regex matching for multiple solutions is not typically much slower than a single solution.

from scallion.

@zero10 I would agree that looks like an issue. test2 should take about twice as long as test[23]. @zero10 If you want to take a shot at fixing it I will review. If not I will take a look at it when I am back in the US.

from scallion.

@richardklafter I was reviewing this further and now feel my issue is separate from the one raised by @agustinx19 in this issue. I will open a new one for the bug I would like addressed.

EDIT:
I have opened a new issue for my problem, #65

I have submitted pull request #66 to correct this issue

from scallion.

Pull request #66 was merged the issue should be fixed if you build from source.

from scallion.