awebneck/savage — pull request #2 by MatmaRex (2011-12-02 10:36:22+00:00)
Title: A bug and an improvement

1. Savage didn't support scientific notation in coordinates (eg 2e-5). This was a simple fix, I just fixed some regexes, and added a test.

2. Due to recursion in def build_direction Savage would sometimes blow up with SystemStackError. I changed that function, so it iteratively parses all coordinates in once call. (I'm also pretty sure it got a little bit faster, although I didn't do any benchmarks - I removed some ugly string munging.) I didn't add any tests here, since for some unexplainable reason it didn't explode in tests, but did in real life - here's a reduced example: https://gist.github.com/1422746, if it behaves "correctly" for you, you can just copy it verbatim.