Is `VisitBaseType` internal field used consistently?
还没有人认领这个 Issue。
评估
调研方向
从 TSqlFragmentVisitor 开始，比较 Visit 和 ExplicitVisit 的实现，以 DeclareCursorStatement 及其 VisitBaseType 分支作为具体示例。检查相应的 SQL-fragment visitor 方法中是否存在相反的条件，并记录该行为是否是有意为之，以及应如何解释 VisitBaseType。
由索引模型根据 Issue 内容生成。
描述
While diving deep into debugging my app I accidentally have noticed this: TSqlFragmentVisitor final code contains many blocks which check of VisitBaseType internal field value and if it is true then Visit method for base types is called, but for some sql-fragment classes Visit method implementation has this check negated and the behavior is reversed:
// Summary:
// Visitor for DeclareCursorStatement
public virtual void Visit(DeclareCursorStatement node)
{
if (!VisitBaseType) <<<--- here
{
Visit((TSqlFragment)node);
}
}
//
// Summary:
// Explicit Visitor for DeclareCursorStatement
public virtual void ExplicitVisit(DeclareCursorStatement node)
{
if (VisitBaseType)
{
Visit((TSqlStatement)node);
Visit((TSqlFragment)node);
}
Visit(node);
node.AcceptChildren(this);
}
Is this expected behavior? If so, please clarify what was the intent, how VisitBaseType should be understood.
- 主要语言
- GAP
- 星标
- 277
- 派生
- 43
- 平均合并
- 6 天 17 小时
- 30 天内合并 PR
- 3
贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
microsoft/SqlScriptDOM 的其他 Issue
-
难度 2/5 1-3 小时 新手友好度 84/100
microsoft/SqlScriptDOM#228 ·
-
难度 2/5 1-3 小时 新手友好度 62/100
microsoft/SqlScriptDOM#183 ·
-
难度 4/5 3-5 天 新手友好度 56/100
microsoft/SqlScriptDOM#226 · 1 个 reaction ·
-
难度 4/5 3-5 天 新手友好度 48/100
microsoft/SqlScriptDOM#225 ·
-
难度 3/5 1-2 天 新手友好度 58/100
microsoft/SqlScriptDOM#224 ·
查看 microsoft/SqlScriptDOM 的全部 Issue
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 75/100
-
难度 1/5 1 小时以内 新手友好度 95/100
-
难度 2/5 1-3 小时 新手友好度 88/100
-
难度 2/5 1-3 小时 新手友好度 88/100
-
难度 2/5 1-3 小时 新手友好度 88/100
oxc-project/oxc#26944 ·