Int division `bottom / (-1)` exceeds 32bit integers range
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 57/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Tranquilo
- Stack tecnológico
- javascript
- Área
- backend
Línea de trabajo
Start by locating the JavaScript implementation of EuclideanRing integer division and compare it with the quot implementation in purescript-integers, which the issue says normalizes with | 0. Check the relevant division tests and add coverage for bottom / (-1); done when the result stays within the signed 32-bit Int range.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
I found a potential bug in EuclideanRing.
In PureScript, Int is defined as a signed 32-bit integer, but evaluating bottom / (-1) results in 2147483648, which exceeds the 32-bit range.
I think the JS implementation for integer division should include | 0 normalization to fix this.
cf: This issue doesn't occur with truncating division quot defined in purescript-integers, as it correctly includes | 0.
- Lenguaje dominante
- PureScript
- Estrellas
- 168
- Forks
- 90
- Merge medio
- 3 d 1 h
- PR fusionados (30 d)
- 1
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Sin 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 purescript/purescript-prelude
-
Int bottom is not safeAbierto
Dificultad 3/5 1-2 días Aptitud para principiantes 32/100
purescript/purescript-prelude#312 · 5 comentarios ·
-
add liftCompare & friendsAbierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
purescript/purescript-prelude#310 · 5 comentarios ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 24/100
purescript/purescript-prelude#308 · 2 comentarios ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
purescript/purescript-prelude#307 · 1 comentario · 1 reacción ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
purescript/purescript-prelude#305 · 5 comentarios ·
Todos los issues de purescript/purescript-prelude
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
-
remove the use of `forget`Abiertogood first issue help wanted
Dificultad 2/5 1-3 horas Aptitud para principiantes 80/100
Los mantenedores suelen responder en 1 día
-
bug good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
milind-soni/OpenMausBot#5829 ·
Los mantenedores suelen responder en 1 día