Skip to content

Commit 11b3456

Browse files
committed
remove unneeded wrapper
1 parent 7044f78 commit 11b3456

File tree

1 file changed

+4
-12
lines changed

1 file changed

+4
-12
lines changed

lib/Jsrt/Core/JsrtCore.cpp

Lines changed: 4 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1394,31 +1394,23 @@ CHAKRA_API JsAllocRawData(_In_ JsRuntimeHandle runtimeHandle, _In_ size_t sizeIn
13941394

13951395
CHAKRA_API JsIsCallable(_In_ JsValueRef object, _Out_ bool *isCallable)
13961396
{
1397-
return ContextAPIWrapper<JSRT_MAYBE_TRUE>([&](Js::ScriptContext *scriptContext, TTDRecorder& _actionEntryPopper) -> JsErrorCode {
1398-
PERFORM_JSRT_TTD_RECORD_ACTION_NOT_IMPLEMENTED(scriptContext);
1397+
return ContextAPINoScriptWrapper_NoRecord([&](Js::ScriptContext *scriptContext) -> JsErrorCode {
13991398
VALIDATE_INCOMING_OBJECT(object, scriptContext);
14001399
PARAM_NOT_NULL(isCallable);
14011400

1402-
Js::RecyclableObject * instance = Js::VarTo<Js::RecyclableObject>(object);
1403-
AssertMsg(scriptContext->GetThreadContext()->IsScriptActive(), "Caller is expected to be under ContextAPIWrapper!");
1404-
1405-
*isCallable = Js::JavascriptConversion::IsCallable(instance);
1401+
*isCallable = Js::JavascriptConversion::IsCallable(object);
14061402

14071403
return JsNoError;
14081404
});
14091405
}
14101406

14111407
CHAKRA_API JsIsConstructor(_In_ JsValueRef object, _Out_ bool *isConstructor)
14121408
{
1413-
return ContextAPIWrapper<JSRT_MAYBE_TRUE>([&](Js::ScriptContext *scriptContext, TTDRecorder& _actionEntryPopper) -> JsErrorCode {
1414-
PERFORM_JSRT_TTD_RECORD_ACTION_NOT_IMPLEMENTED(scriptContext);
1409+
return ContextAPINoScriptWrapper_NoRecord([&](Js::ScriptContext *scriptContext) -> JsErrorCode {
14151410
VALIDATE_INCOMING_OBJECT(object, scriptContext);
14161411
PARAM_NOT_NULL(isConstructor);
14171412

1418-
Js::RecyclableObject * instance = Js::VarTo<Js::RecyclableObject>(object);
1419-
AssertMsg(scriptContext->GetThreadContext()->IsScriptActive(), "Caller is expected to be under ContextAPIWrapper!");
1420-
1421-
*isConstructor = Js::JavascriptOperators::IsConstructor(instance);
1413+
*isConstructor = Js::JavascriptOperators::IsConstructor(object);
14221414

14231415
return JsNoError;
14241416
});

0 commit comments

Comments
 (0)