Skip to content

[1195] 移除 os-mingw?:scheme 调用 + glue 绑定 + C++ 函数及死代码 - #4278

Merged
da-liii merged 3 commits into
mainfrom
da/1195/mingw
Aug 9, 2026
Merged

[1195] 移除 os-mingw?:scheme 调用 + glue 绑定 + C++ 函数及死代码#4278
da-liii merged 3 commits into
mainfrom
da/1195/mingw

Conversation

@da-liii

@da-liii da-liii commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

改动

移除 os-mingw? 及其 C++ 对应函数 os_mingw()——项目不支持 MinGW 构建,该谓词在所有代码路径中始终返回 #f,相关分支均为死代码。

Scheme 层(progs + plugins)

  • 9 处 os-mingw? 调用全部移除:(or (os-mingw?) (os-windows?))(os-windows?)(7 处),独立 (os-mingw?)if 坍缩为 else 分支(2 处)
  • glue-symbols.scm 删除 "os-mingw?" 条目

Glue 层

  • glue_lolly.lua 删除 os-mingw?os_mingw 绑定

C++ 层

  • sys_utils.hpp/.cpp 删除 os_mingw() 声明与定义
  • 6 处调用点清理:os_mingw () 全部移除
  • url.cpp 连带删除 heuristic_is_mingw_defaulturl_mingw_default 死代码

不改动

  • OS_MINGW 预处理器宏保留——它是编译期条件,与运行时谓词无关

🤖 Generated with Claude Code

@da-liii
da-liii merged commit a5dbc24 into main Aug 9, 2026
9 checks passed
@da-liii
da-liii deleted the da/1195/mingw branch August 9, 2026 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant