杨必赞
杨必赞
我看这个PR的内容还是你一开始提的方式,直接在csharp的`~JsEnv`里调`PuertsDLL.DestroyInspector`。 而不是在C++的`~JSEngine`,或者说`PuertsDLL.DestroyJSEngine`的时候调用`DestroyInspector` 你现在在C#的`~JsEnv`多调了一次`PuertsDLL.DestroyInspector`,就相当于整个销毁过程里有两段判断inspector存在并销毁的代码。
暂无。可以修改`v8-build`和github action脚本加一个
> 请问开启了WITH_INSPECTOR宏对于性能有影响吗?另外如果使用host_reload对于性能是否有影响?是否可以在线上环境使用? 线上环境不建议以这种形式查问题
刚刚测试的结果是,Action类型传到JS后,是一个C#的NativeObject,而不是直觉上的转成了一个JSFunction。 这里可能有其历史原因,需要点时间调研。 不过最新版本的代码里我给Action的dts声明添加过Invoke方法了。
已经更新1.3.4。
我能在安卓复现你说的问题,但win下使用v12并不能。你能把你v12有问题的版本传一个分支上来吗?
在windows的UnitTest项目加多线程调用时复现,C#侧的THREAD_SAFE对该问题没有作用。 在C++侧添加v8::Locker后确认可以解决该问题。稍晚合入