Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

Is `VisitBaseType` internal field used consistently?

Aperta
#144 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
4/5
Tempo stimato
3-5 giorni
Idoneità per principianti
35/100
Tipo di issue
Bug
Chiarezza
Abbastanza chiara
Stato di attività
Ferma
Stack tecnologico
csharp
Ambito
compilers

Direzione di ricerca

Inizia da TSqlFragmentVisitor e confronta le implementazioni di Visit ed ExplicitVisit, usando DeclareCursorStatement e i relativi rami VisitBaseType come esempio concreto. Controlla se i corrispondenti metodi del visitor dei frammenti SQL contengono la condizione invertita e documenta se il comportamento è intenzionale e come dovrebbe essere interpretato VisitBaseType.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

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.

Lingua principale
GAP
Stelle
277
Fork
43
Merge medio
6g 17h
PR unite (30g)
3

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di microsoft/SqlScriptDOM

Tutte le issue di microsoft/SqlScriptDOM

Issue simili

Altre issue su Compilers

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.