Skip to content

feat(game): game process management - #6864

Open
ToobLac wants to merge 13 commits into
HMCL-dev:mainfrom
ToobLac:game-process-mng
Open

feat(game): game process management#6864
ToobLac wants to merge 13 commits into
HMCL-dev:mainfrom
ToobLac:game-process-mng

Conversation

@ToobLac

@ToobLac ToobLac commented Sep 13, 2026

Copy link
Copy Markdown
Contributor

添加游戏进程管理功能

在打开游戏进程管理界面后,会列出所有当前启动器启动的、正在运行的游戏进程,并可进行打开日志窗口、终止进程等操作。

  • 在没有打开界面时就已经关闭的进程不会出现在列表里
  • 打开界面后退出的进程仍然会显示在界面上并可打开日志窗口,直到刷新或关闭界面

Resolves #5719 Resolves #6848

@github-actions github-actions Bot added the 100+ label Sep 13, 2026

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 Static analysis found the following issues (max count 10):

Comment thread HMCL/src/main/java/org/jackhuang/hmcl/ui/instances/GameProcessPage.java Outdated
@Minecraft269

Minecraft269 commented Sep 13, 2026

Copy link
Copy Markdown

是否可支持在重启实例? 还有打开对应的日志文件(属于是把管理实例的那套东西搬过来了) 还有持久化(现在那个列表是刷新就没了)等等

@ToobLac

ToobLac commented Sep 13, 2026

Copy link
Copy Markdown
Contributor Author

是否可支持在重启实例

也许可做

打开对应的日志文件(属于是把管理实例的那套东西搬过来了)

这个可以考虑

还有持久化(现在那个列表是刷新就没了)

持久化可能会导致oom

@ToobLac
ToobLac marked this pull request as draft September 13, 2026 06:21
@Minecraft269

Copy link
Copy Markdown

还有持久化(现在那个列表是刷新就没了)

持久化可能会导致oom

那实例列表(各种下载列表)为什么没OOM ( )

@ToobLac

ToobLac commented Sep 13, 2026

Copy link
Copy Markdown
Contributor Author

那实例列表(各种下载列表)为什么没OOM ( )

现在一个实例的日志都有可能把 HMCL 刷到卡死,更何况保留一堆实例的日志

@Minecraft269

Copy link
Copy Markdown

现在一个实例的日志都有可能把 HMCL 刷到卡死,更何况保留一堆实例的日志

emmm.... 能不能抛弃(异步)呢?

@ToobLac

ToobLac commented Sep 13, 2026

Copy link
Copy Markdown
Contributor Author

emmm.... 能不能抛弃(异步)呢?

跟异步有什么关系?单纯是日志太多了,内存和计算压力都很大

@Minecraft269

Copy link
Copy Markdown

仅在查看日志的时候加载日志?

@ToobLac

ToobLac commented Sep 13, 2026

Copy link
Copy Markdown
Contributor Author

仅在查看日志的时候加载日志?

那还得从文件加载日志

总之我不会做这个功能,如果要做以后再说

@Minecraft269

Copy link
Copy Markdown

那出了我在提issue bushi

@ToobLac

ToobLac commented Sep 13, 2026

Copy link
Copy Markdown
Contributor Author

还有打开对应的日志文件

事实证明这个也不好做,因为不存在一个方法能准确判定哪个日志文件属于这个进程。而且执行完之后日志文件都是打包到 gzip 里面的

@Minecraft269

Copy link
Copy Markdown

事实证明这个也不好做,因为不存在一个方法能准确判定哪个日志文件属于这个进程。而且执行完之后日志文件都是打包到 gzip 里面的

能不能在文件内注入标识符? (或者放弃这个功能)

@ToobLac
ToobLac marked this pull request as ready for review September 13, 2026 08:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature] 游戏游戏中打开日志窗口 [Feature] 关于已启动的实例进程管理功能的建议

2 participants