method signatures

Abierto
#5 4 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
5/5
Tiempo estimado
Más de una semana
Aptitud para principiantes
25/100
Tipo de issue
Refactorización
Claridad
Necesita aclaración
Estado de actividad
Estancado
Stack tecnológico
rust
Área
data

Línea de trabajo

Comienza leyendo las rutinas encode(), decode() y las correspondientes rutinas _with_offset(), y después examina cómo se comportan sus resultados usize con búferes de origen o destino demasiado pequeños. Compara la implementación desert citada y determina la API y la semántica de errores previstas; se considera terminado cuando las firmas y las rutinas de offset tienen un reemplazo claramente acordado, con pruebas que cubran los fallos por búfer insuficiente.

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

Descripción

encode() and decode() both have failure modes if you pass in a source or destination buffer that is too small but the return type is usize for both encode() and decode().

The _with_offset() routines could likewise be removed, since slices in rust are very cheap.

These are the routines I'm using inline in desert but I considered using this varinteger first:

https://github.com/substack/desert/blob/46789f4af0e98afb80438cb80db573d0d8dbb630/src/lib.rs#L1020-L1054

Lenguaje dominante
Rust
Estrellas
15
Forks
2
Métricas de merge de PR
Sin PR fusionados en 30 d

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 datrs/varinteger

Todos los issues de datrs/varinteger

Issues similares

Más issues de Rust

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.