registerDecls swallows its options parameter
还没有人认领这个 Issue。
评估
- 难度
- 5/5
- 预计耗时
- 一周以上
- 新手友好度
- 35/100
- Issue 类型
- 重构
- 描述清晰度
- 需要澄清
- 活跃度
- 冷清
- 技术栈
- zig
调研方向
从 src/register_decls.zig 开始,检查 registerDecls 及其对 createCallback 的调用,然后查看 issue #59 的上下文。确定是否应转发选项,或将其移除/记录文档,并考虑对公共 API 的影响;完成的标准是 registerDecls 不再悄悄丢弃一个已接受的参数。
由索引模型根据 Issue 内容生成。
描述
Context
Noted while reviewing #59. registerDecls(decls, options) accepts a comptime options parameter but discards it (_ = options;), while internally passing a hardcoded .{} as the options to createCallback. The parameter is dead weight in a public signature — callers can pass anything and it silently does nothing.
Proposal
Either:
- forward
options(or a per-decl subset of it) tocreateCallbackso callers can use arg/return hints (ArgHint, returns hints) withregisterDecls, or - remove/document the parameter as reserved until there is a concrete design for it.
Ref: src/register_decls.zig
- 主要语言
- Zig
- 星标
- 4
- 派生
- 4
- PR 合并指标
- 30 天内没有已合并 PR
贡献指南
这个仓库没有索引到贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
ChainSafe/zapi 的其他 Issue
-
难度 1/5 1 小时以内 新手友好度 92/100
-
Type: Maintenance
难度 1/5 1 小时以内 新手友好度 76/100
-
clearer errors 未关闭
难度 2/5 1-3 小时 新手友好度 68/100
-
难度 2/5 1-3 小时 新手友好度 76/100
-
难度 2/5 1-3 小时 新手友好度 72/100
相似的 Issue
-
bug
难度 2/5 1-3 小时 新手友好度 82/100
-
area/profiles comp/cli comp/dashboard P2 sweeper:risk-session-state type/bug
难度 2/5 1-3 小时 新手友好度 82/100
NousResearch/hermes-agent#122365 ·
-
难度 2/5 1-3 小时 新手友好度 78/100
vllm-project/production-stack#1105 ·
-
bug priority:low
难度 2/5 1-3 小时 新手友好度 84/100
CyberAgent/psd2svg#430 ·
-
难度 2/5 1-3 小时 新手友好度 75/100