src: fix -Wunused-result compiler warning

bnoordhuis · Italo A. Casas · commit af06f62e3567 · 2017-02-14T11:52:24.000-05:00
Fix a warning that was introduced in commit 67af1ad ("src: refactor CopyProperties to remove JS") from a few days ago. This particular change was suggested by me, mea culpa. Fixes the following warning: ../src/node_contextify.cc:151:13: warning: ignoring return value of function declared with warn_unused_result attribute [-Wunused-result] sandbox_obj->DefineProperty(context, key, *desc); PR-URL: #11197 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
diff --git a/src/node_contextify.cc b/src/node_contextify.cc
@@ -148,7 +148,7 @@ class ContextifyContext {
             desc->set_enumerable(desc_vm_context
                 ->Get(context, env()->enumerable_string()).ToLocalChecked()
                 ->BooleanValue(context).FromJust());
-            sandbox_obj->DefineProperty(context, key, *desc);
+            CHECK(sandbox_obj->DefineProperty(context, key, *desc).FromJust());
         };
 
         if (is_accessor) {
