[UE] Bug: JsEnvImpl的构造函数在纯v8的环境下没有正确处理传入的InFlags

[BlurryLight]

前置阅读 | Pre-reading

• 文档/Doc
• Issues
• FAQ

Puer的版本 | Puer Version

1.0.5

UE的版本 | UE Version

5

发生在哪个平台 | Platform

Editor(win)

错误信息 | Error Message

No response

问题重现 | Bug reproduce

JsEnvImpl的构造函数在纯v8的环境下没有正确处理传入的InFlags。

puerts/unreal/Puerts/Source/JsEnv/Private/JsEnvImpl.cpp

Lines 345 to 360 in 6256231

	if (!InFlags.IsEmpty())
	{
	#if !defined(WITH_NODEJS) && !defined(WITH_QUICKJS)
	TArray<FString> FlagArray;
	InFlags.ParseIntoArray(FlagArray, TEXT(" "));
	for (auto& Flag : FlagArray)
	{
	static FString Max_Old_Space_Size_Name(TEXT("--max-old-space-size="));
	if (Flag.StartsWith(Max_Old_Space_Size_Name))
	{
	size_t Val = FCString::Atoi(*Flag.Mid(Max_Old_Space_Size_Name.Len()));
	CreateParams.constraints.set_max_old_generation_size_in_bytes(Val * 1024 * 1024);
	}
	}
	#else
	v8::V8::SetFlagsFromString(TCHAR_TO_UTF8(*InFlags));

第359行这个#else会导致 纯v8环境下无法走进去V8::SetFlagsFromString。

[chexiongsheng]

都想不起为啥这样写了，可能是笔误吧。。

[chexiongsheng]

你试了直接去掉else，endif提前生效吗？

[BlurryLight]

我是直接把#else那边替换成了#endif。感觉是个笔误。

[chexiongsheng]

应该不能都调用v8::V8::SetFlagsFromString，这是全局的，而前面那段主要是希望只对单虚拟机生效。
这里直接去掉v8::V8::SetFlagsFromString比较合适，可以另外加个静态函数作为全局生效的接口。