Rename VisitorMut::visit to VisitorMut::visit_mut

Abierto
#2,067 0 comentarios 1 reacción 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
58/100
Tipo de issue
Refactorización
Claridad
Bastante claro
Estado de actividad
Estancado
Stack tecnológico
rust
Área
backend-api-design

Línea de trabajo

Busca en el repositorio el trait VisitorMut y su método visit; después, sigue las implementaciones y los sitios de llamada que muestran esas coincidencias. Cambia el nombre del método de forma coherente y verifica que Visitor y VisitorMut puedan usarse en el mismo contexto sin el error de trait bound indicado; ejecuta las pruebas o comprobaciones de Rust existentes relevantes.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

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:

        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`
Lenguaje dominante
Rust
Estrellas
3.5k
Forks
774
Merge medio
3 d 4 h
PR fusionados (30 d)
19

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de apache/datafusion-sqlparser-rs

Todos los issues de apache/datafusion-sqlparser-rs

Issues similares

Más issues de Rust

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.