[Suggestion] Prevent usage of openSync
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 42/100
- Tipo de issue
- Refactorización
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- javascript
- Área
- operating-systems
Línea de trabajo
Empieza en lib/index.js rastreando cómo se utilizan fs.openSync, this.fd y fs.createReadStream(null, this). Compara ese flujo con pasar this.device directamente a createReadStream y, después, verifica que los eventos de entrada de los dispositivos Linux se sigan leyendo correctamente y que no se pierda ningún comportamiento necesario del descriptor de archivo.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Are the following lines in lib/index.js really necessary?
this.fd = fs.openSync(this.device, this.flags);
this.input = fs.createReadStream(null, this);
At least, you do not check for a valid file descriptor first. So why not forward the device string directly to the createReadStream function instead of null?
this.input = fs.createReadStream(this.device, this);
- Lenguaje dominante
- JavaScript
- Estrellas
- 65
- Forks
- 22
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de lsongdev/input-event
-
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
lsongdev/input-event#21 · 3 comentarios ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 30/100
lsongdev/input-event#16 · 1 comentario ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 25/100
lsongdev/input-event#13 · 1 comentario ·
-
help wanted
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
lsongdev/input-event#10 · 1 reacción ·
-
help wanted
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
lsongdev/input-event#6 · 2 comentarios ·
Todos los issues de lsongdev/input-event
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 79/100
SACGF/variantgrid#2341 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día
-
Unsampled pageload span is profiled when profileLifecycle is 'trace' (browser UI profiling)Posiblemente ocupada @Jxxunnn la tomó hoy. AbiertoBrowser Waiting for: Product Owner
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
getsentry/sentry-javascript#25497 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
[External Plugin]: rostrumAbiertoexternal-plugin needs-review:HIGH ready-for-review
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
github/awesome-copilot#5162 · 3 comentarios ·
Los mantenedores suelen responder en 1 día