On the "safety" of escape_into.

Abierto
#73 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
5/5
Tiempo estimado
Más de una semana
Aptitud para principiantes
30/100
Tipo de issue
Nueva funcionalidad
Claridad
Bastante claro
Estado de actividad
Tranquilo
Stack tecnológico
rust
Área
performance

Línea de trabajo

El issue solo menciona el punto de entrada escape_into y no indica archivos ni pruebas. Empieza localizando su implementación y revisando el contrato actual de la API; después, compara las firmas propuestas de escape_append. Para darlo por terminado, es necesario haber definido la API y el diseño de seguridad antes de la implementación.

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

Descripción

I am talking about

pub fn escape_into(value: &str, dst: &mut Vec<u8>) {

This thing is pure unsafe semantically, a clear UB. IMO it should be deprecated and an alternative should be recommended:

pub unsafe fn escape_append(value: &str, dst: *mut u8, len: usize) -> usize { // or *mut u8

This would be

  • Fair. One should not promise what he's not going to deliver.
  • Performant and actually more handy.
Lenguaje dominante
Rust
Estrellas
21
Forks
6
Merge medio
10 h 47 min
PR fusionados (30 d)
13

Preparar el entorno

Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.

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 napi-rs/escape-simd

Todos los issues de napi-rs/escape-simd

Issues similares

Más issues de Rust

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.