Skip to content

Conversation

saghul
Copy link
Contributor

@saghul saghul commented Sep 13, 2024

We don't use it.

@bnoordhuis
Copy link
Contributor

Suggestion:

diff --git a/doc/quickjs.texi b/doc/quickjs.texi
index c335eeb..6a4ba10 100644
--- a/doc/quickjs.texi
+++ b/doc/quickjs.texi
@@ -198,16 +198,8 @@ can be installed in the QuickJS source directory with:
 
 @example
 git clone https://github.com/tc39/test262.git test262
-cd test262
-patch -p1 < ../tests/test262.patch
-cd ..
 @end example
 
-The patch adds the implementation specific @code{harness} functions
-and optimizes the inefficient RegExp character classes and Unicode
-property escapes tests (the tests themselves are not modified, only a
-slow string initialization function is optimized).
-
 The tests can be run with
 @example
 make test2

That whole section is probably wrong-ish because we have tc39/test262 as a git submodule.

@saghul
Copy link
Contributor Author

saghul commented Sep 13, 2024

Good point, I'll adjust it!

We don't use it.
@saghul saghul merged commit 9bb8a68 into master Sep 13, 2024
17 checks passed
@saghul saghul deleted the saghul-patch-3 branch September 13, 2024 19:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants