#74 Fix #75

merged 2 commits into from Mar 26, 2012

2 participants


I wanted to get rid of the regex but the alternative of parsing the attribute string manually was slower and more complicated.

Moved the regular expressions to the top of the file to avoid recompiling them each time the function is called. Ran it against the hotshot profiler and it was too slow for my liking. Also split them up to make it more readable.

unknown added some commits Mar 25, 2012
unknown Fixes #74: Regex to quote keys no longer messes with attribute values…
…. Also, precompiling regex for speed and readability
unknown (#74). Small typo 9cd367c
@jessemiller jessemiller merged commit a70cbbf into jessemiller:master Mar 26, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment