domains: properly check if domains are being used

trevnorris · trevnorris · commit e3c5019ac70c · 2013-08-06T15:42:47.000-07:00
process.domain is almost never just undefined, so it was setting the
object property unnecessarily.
diff --git a/src/req_wrap.h b/src/req_wrap.h
@@ -47,7 +47,7 @@ class ReqWrap {
                                     ->ToObject()
                                     ->Get(domain_symbol);
 
-      if (!domain->IsUndefined()) {
+      if (domain->IsObject()) {
         object->Set(domain_symbol, domain);
       }
     }

Original file line number	Diff line number	Diff line change
`@@ -47,7 +47,7 @@ class ReqWrap {`
`47`	`47`	`->ToObject()`
`48`	`48`	`->Get(domain_symbol);`
`49`	`49`
`50`		`- if (!domain->IsUndefined()) {`
	`50`	`+ if (domain->IsObject()) {`
`51`	`51`	`object->Set(domain_symbol, domain);`
`52`	`52`	`}`
`53`	`53`	`}`