Skip to content
This repository has been archived by the owner on Aug 31, 2018. It is now read-only.

Commit

Permalink
src: remove unused warning in node_contextify
Browse files Browse the repository at this point in the history
PR-URL: nodejs/node#16408
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com>
  • Loading branch information
targos authored and addaleax committed Oct 26, 2017
1 parent 914ab9c commit abf81ae
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/node_contextify.cc
Original file line number Diff line number Diff line change
Expand Up @@ -440,7 +440,8 @@ class ContextifyContext {
desc_for_sandbox->set_configurable(desc.configurable());
}
// Set the property on the sandbox.
sandbox->DefineProperty(context, property, *desc_for_sandbox);
sandbox->DefineProperty(context, property, *desc_for_sandbox)
.FromJust();
};

if (desc.has_get() || desc.has_set()) {
Expand Down

0 comments on commit abf81ae

Please sign in to comment.