Prhub

#20668 refactor: Remove dead code from utils/common.py

原始 PR 作者 alphabetc1 合并时间 2026-03-22 12:54 文件变更 1 提交数 2 评论 3 代码增减 +0 / -155

执行摘要

删除 utils/common.py 中的死代码,减少噪音并提升可维护性。

utils/common.py 中累积了多个没有调用者的函数,例如 get_float_env_var、decode_video_base64 等,这些函数在之前的 PR(如 #14501、#5888)中被重构或移除调用点后成为死代码。移除它们可以减少代码噪音,保持模块的简洁性和可维护性,避免未来维护时的混淆。

该 PR 变更简单直接,适合快速浏览以了解代码清理的最佳实践。对于新贡献者,可以学习如何识别和移除死代码;无需精读,除非关注代码维护流程。

讨论亮点

review 中仅有一个 bot 评论(gemini-code-assist[bot])表示支持,认为移除死代码是好的重构实践,变更直接且无问题。没有其他讨论或争议,变更被迅速接受。

实现拆解

在唯一变更文件 python/sglang/srt/utils/common.py 中,删除了以下未使用的符号:函数 get_float_env_var、decode_video_base64、pytorch_profile、dump_to_file、maybe_torch_compile、is_triton_3,变量 step_counter,以及相关导入语句 from torch.profiler import ...。通过全仓库 grep 验证这些符号在其他 *.py 文件中无引用。

文件 模块 状态 重要度
python/sglang/srt/utils/common.py sglang/srt/utils modified 4.0

关键符号

get_float_env_var decode_video_base64 pytorch_profile dump_to_file maybe_torch_compile is_triton_3

分析完成后,这里会展示 LLM 生成的相对完整源码片段和详细注释。

评论区精华

代码清理实践 设计

gemini-code-assist[bot] 评论认为移除死代码是好的重构实践,变更直接且无问题。

结论:变更被接受,无争议。 · 已解决

风险与影响

风险极低:删除的是已验证无调用者的死代码,不会影响现有功能。唯一潜在风险是如果未来有代码错误地引用了这些符号,但已通过全仓库 grep 验证无引用,确保了安全性。无回归、性能、安全或兼容性问题。

对用户和系统无直接影响,仅提升代码库的整洁度。对团队来说,减少了不必要的代码维护,提高了模块的可读性和可维护性,影响范围限于 utils/common.py 模块。

无风险

关联 Issue

未识别关联 Issue

当前没有检测到明确关联的 Issue 链接,后续同步到相关引用后会出现在这里。

完整报告

参与讨论