haskell / haskell/bytestring

Audit `foreign import ccall` types

Abierto
#653 0 comentarios 0 reacciones 1 asignado Reclamado por @clyring Ver en GitHub
bug
Lenguaje dominante
Haskell
Estrellas
301
Forks
144
Merge medio
7 d 22 h
PR fusionados (30 d)
1

Descripción

In [this comment](https://github.com/haskell/bytestring/pull/631#discussion_r1451112873) it was noticed that we have a mismatch between the actual C prototype of `sbs_elem_index` and the foreign import in our Haskell code:

```c
ptrdiff_t
sbs_elem_index(const void *s,
int c,
size_t n)
{ // ...
```

```haskell
foreign import ccall unsafe "static sbs_elem_index"
c_elem_index :: ByteArray# -> Word8 -> CSize -> IO CPtrdiff
```

Notice that for the second argument we have `Word8` on the Haskell side and `int` on the C side.

I will audit our foreign imports for similar mistakes.

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Evaluación

Este issue todavía no se ha evaluado.

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.