instrumentisto / medea

[Closed] Medea media server
Mozilla Public License 2.0
23 stars 3 forks source link

Use ptr::NonNull in Jason FFI API #202

Closed alexlapa closed 3 years ago

alexlapa commented 3 years ago

Related to #183

Synopsis

ptr::NonNull можно спокойно использовать как замену сырым указателям. Никаких дополнительных гарантий это нам не дает, но:

  1. Избавляемся от избыточных анврапов.
  2. Технически, нулевым указателям у нас взяться неоткуда. Будет правильно это выразить через типы.

Solution

Use ptr::NonNull in Jason FFI API.

Еще поменял std::os::raw::c_char на libc::c_char. На практике разницы это никакой не дает. Но, учитывая, что мы тут делаем FFI, будет малость корректнее в подобных ситуациях предпочитать типы из libc.

Checklist

alexlapa commented 3 years ago

FCM:

Use ptr::NonNull in Jason FFI API (#202)