need to account for new regexp behavior

VERIFIED FIXED

Status

Other Applications
Venkman JS Debugger
VERIFIED FIXED
14 years ago
13 years ago

People

(Reporter: Robert Ginda, Assigned: Robert Ginda)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

14 years ago
The checkin for bug 85721, which fixed bug 123437, changes regexp.match in a way
that breaks venkman.  The same type of changes were required in chatzilla in bug
223247.
(Assignee)

Comment 1

14 years ago
Created attachment 134053 [details] [diff] [review]
patch
(Assignee)

Updated

14 years ago
Attachment #134053 - Flags: approval1.6a?

Comment 2

14 years ago
Comment on attachment 134053 [details] [diff] [review]
patch

a=asa (on behalf of drivers) for checkin to 1.6alpha
Attachment #134053 - Flags: approval1.6a? → approval1.6a+

Comment 3

14 years ago
Anyone planning to land this for 1.6?
(Assignee)

Comment 4

14 years ago
this has already benn checked in
Status: NEW → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED

Comment 5

14 years ago
> this has already benn checked in
Not as far as I can see, it hasn't. Am I looking in the wrong place?

For example:
http://bonsai.mozilla.org/cvslog.cgi?
file=mozilla/extensions/venkman/resources/content/command-manager.js
(last checkin May 15, 2003)
(Assignee)

Comment 6

14 years ago
Comment on attachment 134053 [details] [diff] [review]
patch

oops, my bad.  I thought this was the chatzilla version.  Don;t know how I
forgot to check this in.
Attachment #134053 - Flags: approval1.6a+ → approval1.6?

Comment 7

14 years ago
Comment on attachment 134053 [details] [diff] [review]
patch

a=asa (on behalf of drivers) for checkin to Mozilla 1.6
Attachment #134053 - Flags: approval1.6? → approval1.6+

Updated

14 years ago
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Assignee)

Comment 8

14 years ago
checked in for real this time.
Status: REOPENED → RESOLVED
Last Resolved: 14 years ago14 years ago
Resolution: --- → FIXED
(Assignee)

Comment 9

14 years ago
almost forgot... thanks for the reminder malcolm :)

Comment 10

14 years ago
No problem. Verified fixed per bonsai.
Status: RESOLVED → VERIFIED
Product: Core → Other Applications
You need to log in before you can comment on or make changes to this bug.