[wp-trac] [WordPress Trac] #17657: Shortcode regex doesn't allow for hyphens in shortcode name

WordPress Trac wp-trac at lists.automattic.com
Fri Jul 20 13:38:29 UTC 2012


#17657: Shortcode regex doesn't allow for hyphens in shortcode name
-------------------------------------+--------------------
 Reporter:  sivel                    |       Owner:
     Type:  defect (bug)             |      Status:  new
 Priority:  normal                   |   Milestone:  3.5
Component:  Shortcodes               |     Version:  2.9.2
 Severity:  normal                   |  Resolution:
 Keywords:  has-patch needs-testing  |
-------------------------------------+--------------------

Comment (by kovshenin):

 Replying to [comment:16 nacin]:

 Sorry, didn't include the add_shortcode() calls in my paste. I altered the
 code to include some dummy text (18k characters) and four shortcodes which
 were used a total of 40 times inside the dummy text. I ran ten tests with
 the original code, and ten tests with the patched code, but still, can't
 see the performance difference you're talking about. Apparently I'm doing
 something wrong :)

 {{{
 // Original total: 181.271103, average: 18.127110, min: 13.958034, max:
 24.967603.
 // Patched total:  182.409497, average: 18.240950, min: 13.885041, max:
 25.773981.
 }}}

 Here's the test I ran: https://gist.github.com/3150734

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


More information about the wp-trac mailing list