212395 - Bad Flag field assignment in javascript.regexp.NativeRegExp

Status: VERIFIED FIXED

(Rhino Graveyard :: Core, defect)

Description

[Olivier Oeuillot]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.4) Gecko/20030624
Build Identifier: 

org.mozilla.javascript.regexp.NativeRegExp


	public void init(Context cx, Scriptable scope, String str, String global,
boolean flat) {


.....

this.parenCount = state.parenCount;
this.flags = flags;  <<<<<<<< NO effect !
this.lastIndex = 0;

....
}


Solution ? :

	public void init(Context cx, Scriptable scope, String str, String global,
boolean flat) {
		this.source = str.toCharArray();
		int length = str.length();
		int flags = 0;   <<<<< int declaration !




Reproducible: Always

Steps to Reproduce:
1.
2.
3.

Comment 1

[Phil Schwartau]

Assigning this one to Roger -

Comment 2

[Igor Bukanov]

Attachment: The above suggestion as a patch

Comment 3

[Igor Bukanov]

I committed the fix.

Comment 4

[Phil Schwartau]

Rubber-stamp vrfy -

Comment 5

[Igor Bukanov]

Trageting as resolved against 1.5R5