Is `VisitBaseType` internal field used consistently?
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 35/100
Línea de trabajo
Comienza en TSqlFragmentVisitor y compara las implementaciones de Visit y ExplicitVisit, usando DeclareCursorStatement y sus ramas de VisitBaseType como ejemplo concreto. Comprueba si las correspondientes funciones visitor de fragmentos SQL tienen la condición invertida y documenta si el comportamiento es intencionado y cómo debería interpretarse VisitBaseType.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- GAP
- Estrellas
- 277
- Forks
- 43
- Merge medio
- 6 d 17 h
- PR fusionados (30 d)
- 3
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de microsoft/SqlScriptDOM
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
microsoft/SqlScriptDOM#228 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
microsoft/SqlScriptDOM#183 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 56/100
microsoft/SqlScriptDOM#226 · 1 reacción ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
microsoft/SqlScriptDOM#225 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 58/100
microsoft/SqlScriptDOM#224 ·
Todos los issues de microsoft/SqlScriptDOM
Issues similares
-
todo:perf
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
-
StUnhex.number() misformats Long.MIN_VALUE as scientific notation instead of a plain integer Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
objectionary/eo#8894 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
objectionary/phie#154 ·
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
objectionary/jeo-maven-plugin#1774 ·
-
Minor breakage w/ LLVM 7: `test_llvm.cpp: error: cannot convert 'llvm::Module' to 'llvm::Module*'` Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100