#106 ✓inactive
disccomp

String#strip performance

Reported by disccomp | August 25th, 2008 @ 08:07 AM

The regex currently in use does not perform as well as several other solutions.

Analysis: http://groups.google.com/group/p...

Discussion: http://groups.google.com/group/p...

Comments and changes to this ticket

  • John-David Dalton

    John-David Dalton August 25th, 2008 @ 04:45 PM

    • Tag set to function, needs_patch, string

    Currently strip doesn't remove newline characters as well.

  • John-David Dalton

    John-David Dalton August 25th, 2008 @ 04:45 PM

    • State changed from “new” to “enhancement”
    • Tag changed from function, needs_patch, string to function, needs_benchmarks, needs_patch, needs_tests, string
  • Xanadu

    Xanadu September 26th, 2008 @ 10:45 PM

    I would like to see performance tests on more recent browsers, namely: * Chrome * Firefox 3 * Opera 9.5

  • disccomp

    disccomp September 26th, 2008 @ 11:35 PM

    Yeah, sounds like a great idea. Let us know how that goes.

    On Fri, Sep 26, 2008 at 4:45 PM, Lighthouse support@lighthouseapp.comwrote:

  • Tobie Langel

    Tobie Langel July 23rd, 2009 @ 04:09 AM

    • Tag changed from function, needs_benchmarks, needs_patch, needs_tests, string to function, needs_benchmarks, needs_patch, needs_tests, performance, string
  • Tobie Langel

    Tobie Langel July 24th, 2009 @ 02:07 AM

    • Tag changed from function, needs_benchmarks, needs_patch, needs_tests, performance, string to needs_benchmarks, needs_patch, needs_tests, performance, section:lang

    [not-tagged:"function" tagged:"section:lang" bulk edit command]

  • Tobie Langel

    Tobie Langel July 24th, 2009 @ 02:26 AM

    • Tag changed from needs_benchmarks, needs_patch, needs_tests, performance, section:lang to missing:tests, needs_benchmarks, needs_patch, performance, section:lang

    [not-tagged:"needs_tests" tagged:"missing:tests" bulk edit command]

  • Tobie Langel

    Tobie Langel July 24th, 2009 @ 02:27 AM

    • Tag changed from missing:tests, needs_benchmarks, needs_patch, performance, section:lang to missing:benchmarks, missing:tests, needs_patch, performance, section:lang

    [not-tagged:"needs_benchmarks" tagged:"missing:benchmarks" bulk edit command]

  • Tobie Langel

    Tobie Langel July 24th, 2009 @ 02:28 AM

    • Tag changed from missing:benchmarks, missing:tests, needs_patch, performance, section:lang to missing:benchmarks, missing:patch, missing:tests, performance, section:lang

    [not-tagged:"needs_patch" tagged:"missing:patch" bulk edit command]

  • Tobie Langel

    Tobie Langel July 24th, 2009 @ 03:36 AM

    • Tag changed from missing:benchmarks, missing:patch, missing:tests, performance, section:lang to missing:benchmarks, missing:patch, needs:tests, performance, section:lang

    [not-tagged:"missing:tests" tagged:"needs:tests" bulk edit command]

  • Tobie Langel

    Tobie Langel July 24th, 2009 @ 03:37 AM

    • Tag changed from missing:benchmarks, missing:patch, needs:tests, performance, section:lang to missing:benchmarks, needs:patch, needs:tests, performance, section:lang

    [not-tagged:"missing:patch" tagged:"needs:patch" bulk edit command]

  • Tobie Langel

    Tobie Langel July 24th, 2009 @ 03:39 AM

    • Tag changed from missing:benchmarks, needs:patch, needs:tests, performance, section:lang to needs:benchmarks, needs:patch, needs:tests, performance, section:lang

    [not-tagged:"missing:benchmarks" tagged:"needs:benchmarks" bulk edit command]

  • T.J. Crowder

    T.J. Crowder November 16th, 2009 @ 04:50 PM

    [responsible:none bulk edit command]

  • Franco Monsalvo

    Franco Monsalvo March 31st, 2010 @ 11:42 PM

    disccomp can you provide us with more up to date benchmarks please?
    Thanks.

  • Franco Monsalvo

    Franco Monsalvo April 11th, 2010 @ 12:53 AM

    • State changed from “enhancement” to “inactive”

    I'm closing this ticket since it has been inactive for quite a while now.
    if you think this is still in issue in the current version of Prototype, please feel free to reopen the ticket.

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.

New-ticket Create new ticket

Create your profile

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

The Prototype JavaScript library.

Shared Ticket Bins

Referenced by

Pages