Commit 622048d
src: prefer to get Environment from Context
We explicitly store the context anyway, and can skip the
extra steps introduced in `Environment::GetCurrent()`.
PR-URL: #26376
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>1 parent 716ec00 commit 622048d
1 file changed
Lines changed: 2 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -333,10 +333,9 @@ inline Local<Value> GetName(Local<Function> fn) { | |||
| 333 | 333 | // execution. | |
| 334 | 334 | void TimerFunctionCall(const FunctionCallbackInfo<Value>& args) { | |
| 335 | 335 | Isolate* isolate = args.GetIsolate(); | |
| 336 | - HandleScope scope(isolate); | ||
| 337 | - Environment* env = Environment::GetCurrent(isolate); | ||
| 336 | + Local<Context> context = isolate->GetCurrentContext(); | ||
| 337 | + Environment* env = Environment::GetCurrent(context); | ||
| 338 | 338 | CHECK_NOT_NULL(env); | |
| 339 | - Local<Context> context = env->context(); | ||
| 340 | 339 | Local<Function> fn = args.Data().As<Function>(); | |
| 341 | 340 | size_t count = args.Length(); | |
| 342 | 341 | size_t idx; | |
0 commit comments