#1677 Import V8's regexp parser code and add build config

Closed
opened 6 months ago by mattatobin · 3 comments

I am going to be following the reference bug except I will be placing the files in the js/src/regexp as I do not want this to eventually overwrite the older implimentation as that would create confused git history.

The configure option is --enable-new-regexp and the relevant define is JS_NEW_REGEXP NOT ENABLE_NEW_REGEXP that Mozilla used. So keep that in mind.


BMO References:

I am going to be following the reference bug except I will be placing the files in the `js/src/regexp` as I do not want this to eventually overwrite the older implimentation as that would create confused git history. The configure option is `--enable-new-regexp` and the relevant define is `JS_NEW_REGEXP` **NOT** `ENABLE_NEW_REGEXP` that Mozilla used. So keep that in mind. ---- BMO References: - [Bug 1592302](https://bugzilla.mozilla.org/show_bug.cgi?id=1592302) - [Bug 1592307](https://bugzilla.mozilla.org/show_bug.cgi?id=1592307) - [Bug 1620020](https://bugzilla.mozilla.org/show_bug.cgi?id=1620020) - [Bug 1624015](https://bugzilla.mozilla.org/show_bug.cgi?id=1624015) - [Bug 1594545](https://bugzilla.mozilla.org/show_bug.cgi?id=1594545)
mattatobin added the
C: Build System
label 6 months ago
mattatobin added the
C: Javascript
label 6 months ago
mattatobin self-assigned this 6 months ago
mattatobin changed title from Import the new V8 code from Mozilla and make it build to Import V8's regexp parser code and make it build 6 months ago
mattatobin changed title from Import V8's regexp parser code and make it build to Import V8's regexp parser code 6 months ago
mattatobin commented 6 months ago
Poster

So as-is it will not compile successfully until more work is done.

So as-is it will not compile successfully until more work is done.
mattatobin changed title from Import V8's regexp parser code to Import V8's regexp parser code and add build config 6 months ago
mattatobin added the
Done
label 6 months ago
mattatobin closed this issue 6 months ago
mattatobin removed the
Done
label 6 months ago
mattatobin reopened this issue 6 months ago
mattatobin commented 6 months ago
Poster

So because Mozilla sabotaged their import we can’t follow the exact progression AND make it build along each step of the way. So I am going to do the 2nd to 5th bugs as parts of this issue.

A new issue will be to make it build. Subsiquent issues will cover hooking it up and adding more functionality.

So because Mozilla sabotaged their import we can't follow the exact progression AND make it build along each step of the way. So I am going to do the 2nd to 5th bugs as parts of this issue. A new issue will be to make it build. Subsiquent issues will cover hooking it up and adding more functionality.
mattatobin commented 6 months ago
Poster

This is as far as we can get by str8 application of the bugs without it impacting other areas. So this is finished.

This is as far as we can get by str8 application of the bugs without it impacting other areas. So this is finished.
mattatobin closed this issue 6 months ago
mattatobin added the
Done
label 6 months ago
Sign in to join this conversation.
No Milestone
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Blocks
#1675 Update to Mozilla's new V8 regexp parser
MoonchildProductions/UXP
Loading…
There is no content yet.