On the "safety" of escape_into.

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

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
5/5
Tempo stimato
Più di una settimana
Idoneità per principianti
30/100
Tipo di issue
Funzionalità
Chiarezza
Abbastanza chiara
Stato di attività
Tranquilla
Stack tecnologico
rust
Ambito
performance

Direzione di ricerca

L’issue nomina solo il punto di ingresso escape_into e non indica file o test. Inizia individuando la sua implementazione e verificando il contratto API attuale, quindi confronta le firme proposte di escape_append; per considerare il lavoro completato, l’API e il design della sicurezza devono essere definiti prima dell’implementazione.

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

Descrizione

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.
Lingua principale
Rust
Stelle
21
Fork
6
Merge medio
10h 47m
PR unite (30g)
13

Preparare l'ambiente

Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.

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

Tutte le issue di napi-rs/escape-simd

Issue simili

Altre issue su Rust

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.