Changeset 1326 in webkit for trunk/JavaScriptCore/kjs/string_object.cpp

Timestamp:

Jun 10, 2002, 1:08:04 PM (23 years ago)

Author:

darin

Message:

JavaScriptCore:

Merged in changes from KDE 3.0.1.

• kjs/collector.cpp:
• kjs/date_object.cpp:
• kjs/function.cpp:
• kjs/internal.cpp:
• kjs/lookup.h:
• kjs/object.cpp:
• kjs/operations.cpp:
• kjs/regexp.cpp:
• kjs/regexp_object.cpp:
• kjs/regexp_object.h:
• kjs/string_object.cpp:
• kjs/testkjs.cpp:
• kjs/ustring.cpp:
• kjs/value.cpp:
• kjs/value.h: Do the merge, and add APPLE_CHANGES as needed to make things compile.

• kjs/date_object.lut.h: Re-generated.

WebCore:

Merged in changes from KDE 3.0.1.

• kwq/kdecore/kurl.h:
• kwq/KWQKURL.mm: (operator==): New, needed by some 3.0.1 code.

• kwq/qt/qpalette.h: Add QPalette::NColorGroups and QColorGroup::NColorRoles.
• kwq/KWQPalette.mm: Simplify by getting rid of "private" indirection. (QPalette::color): New, needed by some 3.0.1 code.
• kwq/KWQColorGroup.mm: Simplify by getting rid of "private" indirection.

• kwq/qt/qstring.h:
• kwq/KWQString.mm: (QConstString::QConstString): Change parameter to const QChar * to match Qt so we compile.

• kwq/qt/qtextedit.h:
• kwq/KWQTextEdit.mm: (QTextEdit::setTabStopWidth): Add unimplemented placeholder, needed by some 3.0.1 code.

• kwq/qt/qvaluelist.h:
• kwq/KWQValueListImpl.h:
• kwq/KWQValueListImpl.mm: Simplify implementation.

• WebCore-tests.exp: Update for changes in QValueList implementation.

• kwq/kdecore/klibloader.h: Add include of <kio/global.h> to make something compile.

• kwq/khtml/java/kjavaappletcontext.h: Add JType and constructor that takes DCOPObject*.

• kwq/kio/global.h: Add KIO::CacheControl.

• khtml/css/css_ruleimpl.cpp:
• khtml/css/css_ruleimpl.h:
• khtml/css/css_stylesheetimpl.cpp:
• khtml/css/css_stylesheetimpl.h:
• khtml/css/css_valueimpl.cpp:
• khtml/css/css_valueimpl.h:
• khtml/css/cssparser.cpp:
• khtml/css/cssparser.h:
• khtml/css/cssstyleselector.cpp:
• khtml/css/cssstyleselector.h:
• khtml/css/html4.css:
• khtml/dom/css_rule.cpp:
• khtml/dom/css_rule.h:
• khtml/dom/css_stylesheet.cpp:
• khtml/dom/css_stylesheet.h:
• khtml/dom/css_value.cpp:
• khtml/dom/dom2_events.cpp:
• khtml/dom/dom2_range.cpp:
• khtml/dom/dom2_range.h:
• khtml/dom/dom2_views.cpp:
• khtml/dom/dom_doc.cpp:
• khtml/dom/dom_element.cpp:
• khtml/dom/dom_exception.h:
• khtml/dom/dom_node.cpp:
• khtml/dom/dom_string.cpp:
• khtml/dom/dom_text.cpp:
• khtml/dom/dom_xml.cpp:
• khtml/dom/html_base.cpp:
• khtml/dom/html_block.cpp:
• khtml/dom/html_document.cpp:
• khtml/dom/html_element.cpp:
• khtml/dom/html_element.h:
• khtml/dom/html_form.cpp:
• khtml/dom/html_head.cpp:
• khtml/dom/html_image.cpp:
• khtml/dom/html_inline.cpp:
• khtml/dom/html_list.cpp:
• khtml/dom/html_misc.cpp:
• khtml/dom/html_object.cpp:
• khtml/dom/html_table.cpp:
• khtml/ecma/kjs_html.cpp:
• khtml/ecma/kjs_proxy.cpp:
• khtml/ecma/kjs_traversal.cpp:
• khtml/ecma/kjs_window.cpp:
• khtml/html/html_baseimpl.cpp:
• khtml/html/html_formimpl.cpp:
• khtml/html/html_miscimpl.h:
• khtml/html/html_objectimpl.cpp:
• khtml/html/html_objectimpl.h:
• khtml/html/htmlparser.cpp:
• khtml/html/htmlparser.h:
• khtml/html/htmltokenizer.cpp:
• khtml/html/htmltokenizer.h:
• khtml/khtml_part.cpp:
• khtml/khtmlpart_p.h:
• khtml/khtmlview.cpp:
• khtml/misc/helper.cpp:
• khtml/misc/khtmllayout.h:
• khtml/misc/loader.cpp:
• khtml/misc/loader.h:
• khtml/misc/loader_client.h:
• khtml/misc/shared.h: Added.
• khtml/rendering/bidi.cpp:
• khtml/rendering/break_lines.cpp:
• khtml/rendering/font.cpp:
• khtml/rendering/render_applet.cpp:
• khtml/rendering/render_applet.h:
• khtml/rendering/render_container.cpp:
• khtml/rendering/render_form.cpp:
• khtml/rendering/render_form.h:
• khtml/rendering/render_frames.cpp:
• khtml/rendering/render_image.cpp:
• khtml/rendering/render_object.cpp:
• khtml/rendering/render_object.h:
• khtml/rendering/render_replaced.cpp:
• khtml/rendering/render_replaced.h:
• khtml/rendering/render_root.cpp:
• khtml/rendering/render_style.cpp:
• khtml/rendering/render_style.h:
• khtml/rendering/render_table.cpp:
• khtml/rendering/render_table.h:
• khtml/rendering/render_text.cpp:
• khtml/rendering/render_text.h:
• khtml/xml/dom2_eventsimpl.h:
• khtml/xml/dom2_rangeimpl.h:
• khtml/xml/dom2_traversalimpl.cpp:
• khtml/xml/dom2_traversalimpl.h:
• khtml/xml/dom2_viewsimpl.h:
• khtml/xml/dom_docimpl.cpp:
• khtml/xml/dom_docimpl.h:
• khtml/xml/dom_elementimpl.h:
• khtml/xml/dom_nodeimpl.cpp:
• khtml/xml/dom_nodeimpl.h:
• khtml/xml/dom_stringimpl.cpp:
• khtml/xml/dom_stringimpl.h:
• khtml/xml/xml_tokenizer.h: Merge in 3.0.1 changes and add APPLE_CHANGES as necessary to get it to compile.

File:

• trunk/JavaScriptCore/kjs/string_object.cpp (modified) (3 diffs)

trunk/JavaScriptCore/kjs/string_object.cpp

@@ -214 +214 @@
     int **ovector = regExpObj->registerRegexp( reg, u );
     UString mstr = reg->match(u, -1, &pos, ovector);
+    regExpObj->setSubPatterns(reg->subPatterns());
     if (a0.isA(StringType))
       delete reg;
@@ -243 +244 @@
         int **ovector = regExpObj->registerRegexp( reg, u );
         UString mstr = reg->match(u, lastIndex, &pos, ovector);
+        regExpObj->setSubPatterns(reg->subPatterns());
+        if (pos == -1)
+          break;
         len = mstr.size();
+        // special case of empty match
+        if (len == 0 && lastIndex > 0) {
+          pos = lastIndex + 1;
+          if (pos > u.size())
+            break;
+        }
         UString rstr(u3);
         bool ok;
@@ -263 +273 @@
         }
         lastIndex = pos + rstr.size();
-        if ( pos != -1 )
-          u = u.substr(0, pos) + rstr + u.substr(pos + len);
+        u = u.substr(0, pos) + rstr + u.substr(pos + len);
         //fprintf(stderr,"pos=%d,len=%d,lastIndex=%d,u=%s\n",pos,len,lastIndex,u.ascii());
-      } while ( global && pos != -1 );
+      } while (global);
 
       result = String(u);