#6965: urlize should be faster
--------------------------------------+-------------------------------------
Reporter: floguy | Owner: andrewbadr
Status: assigned | Milestone:
Component: Template system | Version: SVN
Resolution: | Keywords:
Stage: Unreviewed | Has_patch: 1
Needs_docs: 0 | Needs_tests: 0
Needs_better_patch: 0 |
--------------------------------------+-------------------------------------
Changes (by andrewbadr):
* cc: [EMAIL PROTECTED] (added)
* needs_better_patch: 1 => 0
* component: Uncategorized => Template system
Comment:
I created a patch that skips the regex match for most words. This resulted
in 10x speed improvement on a test set of real posts. The reason for
checking for '@' and ':' on line 98 is because I don't want to change the
behavior for e.g. http://localhost/ even though it's not clear whether
this should be supported or not (I don't think so). The len call was
removed for good measure.
--
Ticket URL: <http://code.djangoproject.com/ticket/6965#comment:10>
Django Code <http://code.djangoproject.com/>
The web framework for perfectionists with deadlines
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at
http://groups.google.com/group/django-updates?hl=en
-~----------~----~----~----~------~----~------~--~---
