Bug Tracker

Modify ↓

Ticket #6868 (closed bug: fixed)

Opened 3 years ago

Last modified 3 years ago

$.trim very expensive on long strings

Reported by: kemayo Owned by:
Priority: undecided Milestone: 1.4.3
Component: unfiled Version: 1.4.2
Keywords: Cc:
Blocking: Blocked by:

Description

I ran $.trim on a ~17kB string, and it caused my browser to ask me if I wanted to stop an unresponsive script.

Change History

comment:1 Changed 3 years ago by dfens 

<html>
  <head>
  <script language="javascript" src="http://ajax.googleapis.com/ajax/libs/jquery/1.4.2/jquery.js"></script>
    <script>
$(document).ready(function() {
  var txt = "   ";
  for (var i =0; i <1700;i++)
  txt+="1234567890";
  $.trim(txt);
  txt += "                          ";
  $(".inner").text(txt);
  alert('done');
});
    </script>
  </head>
  <body>

  <div class="inner">
Hello

</div>
 
  </body>
</html>

works fine

comment:2 Changed 3 years ago by dmethvin

  • Priority set to undecided
  • Status changed from new to closed
  • Resolution set to fixed

Fixed at  http://github.com/jquery/jquery/commit/141ad3c3e21e7734e67e37b5fb39782fe11b3c18

comment:3 Changed 3 years ago by dmethvin

#7477 is a duplicate of this ticket.

Please follow the  bug reporting guidlines and use  jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.

View ↑

Add a comment

Modify Ticket

Action
as closed
Author


E-mail address and user name can be saved in the Preferences.

Attachments ↑
 
Note: See TracTickets for help on using tickets.