Is `VisitBaseType` internal field used consistently?
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 4/5
- Tempo estimado
- 3-5 dias
- Facilidade para iniciantes
- 35/100
Direção de pesquisa
Comece em TSqlFragmentVisitor e compare as implementações de Visit e ExplicitVisit, usando DeclareCursorStatement e seus branches de VisitBaseType como exemplo concreto. Verifique os métodos correspondentes do visitor de fragmentos SQL em busca da condição invertida e documente se o comportamento é intencional e como VisitBaseType deve ser interpretado.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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.
- Linguagem predominante
- GAP
- Estrelas
- 278
- Forks
- 46
- Merge médio
- 9d 23h
- PRs com merge (30d)
- 2
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Tem um modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de microsoft/SqlScriptDOM
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 84/100
microsoft/SqlScriptDOM#228 ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 62/100
microsoft/SqlScriptDOM#183 ·
-
Add a Multiline option for CASE expressions (WHEN/THEN/ELSE on their own lines)Talvez já em andamento @trg-alasdair assumiu há 5 dias. Aberta
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 56/100
microsoft/SqlScriptDOM#226 · 1 reação ·
-
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 48/100
microsoft/SqlScriptDOM#225 ·
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 58/100
microsoft/SqlScriptDOM#224 ·
Todas as issues de microsoft/SqlScriptDOM
Issues semelhantes
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 76/100
rubys/roundhouse#444 ·
Mantenedores costumam responder em até 1 dia
-
vxc prints a debug line '[flat-codegen] emitted module via the flat path' on every compileTalvez já em andamento @YodHeVauHe assumiu hoje. Abertadevex good first issue
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 70/100
kmmbvnr/rank#196 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 74/100
SciML/ModelingToolkit.jl#5255 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
NVIDIA/cuda-quantum#5539 · 2 comentários ·
Mantenedores costumam responder em até 1 dia