haskell / haskell/bytestring

Audit `foreign import ccall` types

Offen
#653 0 Kommentare 0 Reaktionen 1 zugewiesene Person Beansprucht von @clyring Auf GitHub ansehen
bug
Vorherrschende Sprache
Haskell
Sterne
301
Forks
144
Ø Merge
7 T. 22 Std.
Gemergte PRs (30 T.)
1

Beschreibung

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.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.