apache / apache/datafusion-sqlparser-rs

Rename VisitorMut::visit to VisitorMut::visit_mut

Ouverte
#2,067 0 commentaires 1 réaction 0 personnes assignées Voir sur GitHub
Langage dominant
Rust
Étoiles
3.5k
Forks
772
Merge moyen
4 j 9 h
PR mergées (30 j)
17

Description

There're cases we need to use `Visitor::visit` and `VisitorMut::visit` in the same context. Since they're of the same name, compiler would be confused.

For example:

``` Rust
impl Visitor For NoopVisitor { .. }
impl VisitorMut for NoopVisitorMut { .. }

let mut visitor = NoopVisitor {};
let _ = stmt.visit(&mut visitor);

let mut visitor_mut = NoopVisitorMut {};
let _ = stmt.visit(&mut visitor_mut);
```

Would raise the error:
```
the trait bound `NoopVisitorMut: sqlparser::ast::Visitor` is not satisfied
the trait `sqlparser::ast::Visitor` is implemented for `NoopVisitor`
```

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Piste de recherche

Recherchez dans le dépôt le trait VisitorMut et sa méthode visit, puis suivez les implémentations et les sites d’appel indiqués par ces correspondances. Renommez la méthode de manière cohérente et vérifiez que Visitor et VisitorMut peuvent être utilisés dans le même contexte sans l’erreur de contrainte de trait signalée ; exécutez les tests ou vérifications Rust existants pertinents.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
rust
Domaine
backend-api-design
Type d'issue
Refactorisation
Difficulté
2/5
Temps estimé
1-3 heures
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
58/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.