[wp-trac] [WordPress Trac] #10987: Improve wptexturize performance

WordPress Trac wp-trac at lists.automattic.com
Wed Oct 21 17:43:34 UTC 2009


#10987: Improve wptexturize performance
-------------------------+--------------------------------------------------
 Reporter:  johanee      |       Owner:           
     Type:  enhancement  |      Status:  new      
 Priority:  normal       |   Milestone:  2.9      
Component:  Performance  |     Version:  2.9      
 Severity:  major        |    Keywords:  has-patch
-------------------------+--------------------------------------------------

Comment(by johanee):

 Thank you!

 I tested those strings by hand, and the errors exists in current
 wptexturize too.

 The result for these strings are '''identical''' when comparing current
 revision 12083 and patched version.

 It looks like ''test quotes before s'' was #1258 which was the marked as
 duplicate of #4539, which is ticket noted in ''test quotes before
 numbers'' so these at least appear to be known problems. I did not
 immediately find a ticket for ''test quotes'' but there might well be --
 there are a large number of tickets regarding the subtleties in this
 function.

 That was one reason I've tried to keep the actual parsing identical in
 this patch.

 I'd be happy to look some more at this function but I think fixing these
 existing problems should be a separate issue and a separate patch.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/10987#comment:5>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list