This project is archived and is in readonly mode.

#36 ✓invalid
Thomas Aylott

Swap String match for RegExp match

Reported by Thomas Aylott | May 28th, 2008 @ 10:05 AM | in 2.0 (closed)

Originally posted on Trac by digitarald

Original Trac Ticket

Description

Changing RegExp::match to test for 200% faster initialize, fixing #630 and changed switch to simpler related check.

Trac Attachments

http://merb.devjavu.com/attachme...

http://merb.devjavu.com/timeline...

Trac Comments

Changed 4 months ago by digitarald


        
All changes 

                  Changed 4 months ago by digitarald

          attachment
              diff.patch added

All changes 

          attachment
              diff.patch added
        
      

Changed 4 months ago by sleets


        
good, this project need democracy. 

      

Comments and changes to this ticket

  • Tom Occhino

    Tom Occhino September 21st, 2008 @ 04:49 PM

    • Title changed from “Optimized Event.js” to “Swap String match for RegExp match”
    • State changed from “new” to “open”
    • Tag set to core, enhancement, trivial

    fixing the title of this ticket as the scope is outside of Events.js

  • Jan Kassens

    Jan Kassens November 21st, 2009 @ 02:13 PM

    • Tag changed from core, enhancement, trivial to enhancement
    • Assigned user cleared.
    • State changed from “open” to “invalid”

    Should have been fixed at some point. (I didn't see a useless use of String.match)

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile »

Shared Ticket Bins

People watching this ticket

Referenced by

Pages