Skip to content
This repository has been archived by the owner on Oct 15, 2020. It is now read-only.

Commit

Permalink
[Merge chakra-core/ChakraCore@9753783bc9] [MERGE #3634 @xiaoyinl] Rem…
Browse files Browse the repository at this point in the history
…ove undefined functions in ScriptContext.h

Merge pull request #3634 from xiaoyinl:function_cleanup

These member functions are declared in ScriptContext.h, but are not defined in ScriptContext.cpp:
[LeaveScriptStartCore](https://github.com/Microsoft/ChakraCore/search?q=LeaveScriptStartCore), [CleanEvalMapCache](https://github.com/Microsoft/ChakraCore/search?q=CleanEvalMapCache), [AddFunctionBodyToPropIdMap](https://github.com/Microsoft/ChakraCore/search?q=AddFunctionBodyToPropIdMap).

I wonder if we can remove them, since they are not used.
  • Loading branch information
chakrabot authored and kfarnung committed Jan 9, 2018
1 parent c10b885 commit 21fb2cf
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions deps/chakrashim/core/lib/Runtime/Base/ScriptContext.h
Original file line number Diff line number Diff line change
Expand Up @@ -914,7 +914,6 @@ namespace Js
char16 const * url;

void PrintStats();
BOOL LeaveScriptStartCore(void * frameAddress, bool leaveForHost);

void InternalClose();

Expand Down Expand Up @@ -1065,7 +1064,6 @@ namespace Js

template <typename TCacheType>
void CleanDynamicFunctionCache(TCacheType* cacheType);
void CleanEvalMapCache(Js::EvalCacheTopLevelDictionary * cacheType);

template <class TDelegate>
void MapFunction(TDelegate mapper);
Expand Down Expand Up @@ -1201,7 +1199,6 @@ namespace Js
void SetFastDOMenabled();
BOOL VerifyAlive(BOOL isJSFunction = FALSE, ScriptContext* requestScriptContext = nullptr);
void VerifyAliveWithHostContext(BOOL isJSFunction, HostScriptContext* requestHostScriptContext);
void AddFunctionBodyToPropIdMap(FunctionBody* body);

void BindReference(void* addr);

Expand Down

0 comments on commit 21fb2cf

Please sign in to comment.