-
-
Notifications
You must be signed in to change notification settings - Fork 406
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Async/Await parse #836
Async/Await parse #836
Conversation
Codecov Report
@@ Coverage Diff @@
## master #836 +/- ##
==========================================
- Coverage 59.37% 59.03% -0.34%
==========================================
Files 157 164 +7
Lines 10036 10217 +181
==========================================
+ Hits 5959 6032 +73
- Misses 4077 4185 +108
Continue to review full report at Codecov.
|
Benchmark for 8afaa30Click to view benchmark
|
Benchmark for 5ad0affClick to view benchmark
|
Benchmark for fecde2cClick to view benchmark
|
Benchmark for b4b3537Click to view benchmark
|
Benchmark for 0464a7dClick to view benchmark
|
Benchmark for f06c10fClick to view benchmark
|
Benchmark for cf5ebf5Click to view benchmark
|
Benchmark for d56494aClick to view benchmark
|
Benchmark for db5b328Click to view benchmark
|
Benchmark for e4e169eClick to view benchmark
|
Benchmark for a6b1cbdClick to view benchmark
|
Co-authored-by: Halid Odat <halidodat@gmail.com>
Benchmark for 16e97a5Click to view benchmark
|
Benchmark for 819a314Click to view benchmark
|
Benchmark for b2beea1Click to view benchmark
|
Benchmark for 0616ddcClick to view benchmark
|
Benchmark for 1ca177eClick to view benchmark
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I finally got a bit of time to review this code. It's looking very, very good. Check my comments and let's see if it can be improved!
boa/src/syntax/parser/expression/primary/async_function_expression/mod.rs
Outdated
Show resolved
Hide resolved
boa/src/syntax/parser/expression/primary/async_function_expression/mod.rs
Outdated
Show resolved
Hide resolved
boa/src/syntax/parser/expression/primary/async_function_expression/mod.rs
Outdated
Show resolved
Hide resolved
boa/src/syntax/parser/expression/primary/async_function_expression/mod.rs
Outdated
Show resolved
Hide resolved
boa/src/syntax/parser/statement/declaration/hoistable/async_function_decl/mod.rs
Outdated
Show resolved
Hide resolved
boa/src/syntax/parser/statement/declaration/hoistable/async_function_decl/mod.rs
Outdated
Show resolved
Hide resolved
…sion/mod.rs Co-authored-by: Iban Eguia <razican@protonmail.ch>
…nction_decl/mod.rs Co-authored-by: Iban Eguia <razican@protonmail.ch>
Benchmark for eaf55fcClick to view benchmark
|
Benchmark for eaf55fcClick to view benchmark
|
Benchmark for 0a49725Click to view benchmark
|
Benchmark for 0a49725Click to view benchmark
|
Benchmark for 243ee72Click to view benchmark
|
Benchmark for 1a12e2fClick to view benchmark
|
Benchmark for 3aae7e8Click to view benchmark
|
This Pull Request addresses the parse side of #818 .
It changes the following:
The run methods return undefined (rather than an unimplemented! panic) while waiting for an execution implementation.