xfangfang / wiliwili

第三方B站客户端,目前可以运行在PC全平台、PSVita、PS4 、Xbox 和 Nintendo Switch上
https://xfangfang.github.io/wiliwili
GNU General Public License v3.0
3.11k stars 145 forks source link

Dev: move fn mg_timer_add #170

Closed lanytcc closed 1 year ago

lanytcc commented 1 year ago
  1. 我搜了一下文档,mg_timer_add函数调用一次即可,放置在循环里可能会导致严重的问题
  2. 顺便优化了一下onMessage函数调用带来的内存和cpu开销
xfangfang commented 1 year ago

感谢,这个位置我确实忽视了。

xfangfang commented 1 year ago

mg_timer_free 是不是不需要调用,在mongoose_thread结束时调用了 mg_mgr_free, 这个函数内部会直接free掉所有timer。 在 mongoose_thread join 之后貌似就不应该调用 mg_timer_free 了。

xfangfang commented 1 year ago

有点好奇,你是没有本地环境直接在网页上改的程序嘛?

lanytcc commented 1 year ago

虽然不是,但也差不多了( 环境有问题,最多用docker。但一样生成不了本地的compiled_commands image

xfangfang commented 1 year ago

哈哈,原来如此。 不确定这个软件能不能直接用 vs 编译,我之前是在 MSYS2 的 MinGW64 环境中编译的,README中有写相关的内容,感觉还是得本地有个环境,不然太容易出问题了。

zeromake commented 1 year ago

哈哈,原来如此。 不确定这个软件能不能直接用 vs 编译,我之前是在 MSYS2 的 MinGW64 环境中编译的,README中有写相关的内容,感觉还是得本地有个环境,不然太容易出问题了。

……当然是可以用msvc的,当初做uwp的时候就兼容了,我本地都是用xmake + vscode的,xmake导出vs工程应该也可以直接用vs开发。

lanytcc commented 1 year ago

@zeromake 提示包yoga、tweeny等找不到,我该怎么办?

lanytcc commented 1 year ago

@xfangfang 可以合并了,我这边测试没有问题,cpu占用跟正常看视频没有区别了(甚至更低),我给danmaku_core 增加了两个函数,你看一下是否合理,其他都是删减,接口没有发生变化

xfangfang commented 1 year ago

@maye174 那我先合并了自己改一下,还是你把那两处改一下呢?

lanytcc commented 1 year ago

@maye174 那我先合并了自己改一下,还是你把那两处改一下呢?

等会我来改

xfangfang commented 1 year ago

@maye174 那我先合并了自己改一下,还是你把那两处改一下呢?

等会我来改

reinterpret_cast 那个问题我刚刚测试又可以正常用了,可能是其他原因导致的之前的版本那里有问题。所以那里先不用改了~

万分感谢,我再去搞一搞flatpak构建的时候缓存总是缓存不到的问题,这些搞好没啥问题的话,咱就发个新版本,今天真的辛苦啦~

zeromake commented 1 year ago

@zeromake 提示包yoga、tweeny等找不到,我该怎么办?

先注意一下 xmake xrepo 是否添加了 zeromake/xrepo 的源。 如果添加了还没有,就得看看 xmake f -c -vD -y 的具体输出了。