nolan/tts-rs
1
0
Fork 0
mirror of https://github.com/ndarilek/tts-rs.git synced 2024-11-25 11:19:37 +00:00
Code Issues Projects Releases Wiki Activity

FFI: Make all functions with unsafe blocks completely unsafe.

Browse Source 
This is better than using unsafe blocks inside the functions, as that
tells the compiler that the unsafeness won't leak out of the block,
which isn't true in this case as we're dealing with another unsafe
language.
This commit is contained in:
mcb2003 2020-12-12 19:37:44 +00:00
parent ae6e64d973
commit f71529d9b6
1 changed files with 22 additions and 28 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
src/ffi.rs
View File

@ -69,20 +69,18 @@ pub extern "C" fn tts_default() -> *mut TTS {
/// Free the memory associated with a TTS object.
/// If `tts` is a null pointer, this function does nothing.
#[no_mangle]
pub extern "C" fn tts_free(tts: *mut TTS) {
pub unsafe extern "C" fn tts_free(tts: *mut TTS) {
if tts.is_null() {
return;
}
unsafe {
Box::from_raw(tts); // Goes out of scope and is dropped
}
}
/// Returns the features supported by this TTS engine.
/// `tts` must be a valid pointer to a TTS object.
#[no_mangle]
pub extern "C" fn tts_supported_features(tts: *mut TTS) -> Features {
unsafe { tts.as_ref().unwrap().supported_features() }
pub unsafe extern "C" fn tts_supported_features(tts: *mut TTS) -> Features {
tts.as_ref().unwrap().supported_features()
}
/// Speaks the specified text, optionally interrupting current speech.
@ -90,7 +88,7 @@ pub extern "C" fn tts_supported_features(tts: *mut TTS) -> Features {
/// the backend doesn't provide one).
/// Returns true on success, false on error or if `tts` is NULL.
#[no_mangle]
pub extern "C" fn tts_speak(
pub unsafe extern "C" fn tts_speak(
tts: *mut TTS,
text: *const c_char,
interrupt: bool,
@ -99,7 +97,6 @@ pub extern "C" fn tts_speak(
if tts.is_null() {
return true;
}
unsafe {
let text = CStr::from_ptr(text).to_string_lossy().into_owned();
match tts.as_mut().unwrap().speak(text, interrupt) {
Ok(u) => {
@ -116,28 +113,25 @@ pub extern "C" fn tts_speak(
return false;
}
}
}
}
/// Free the memory associated with an `UtteranceId`.
/// Does nothing if `utterance` is NULL.
#[no_mangle]
pub extern "C" fn tts_free_utterance(utterance: *mut UtteranceId) {
pub unsafe extern "C" fn tts_free_utterance(utterance: *mut UtteranceId) {
if utterance.is_null() {
return;
}
unsafe {
Box::from_raw(utterance);
}
}
/// Stops current speech.
/// Returns true on success, false on error or if `tts` is NULL.
#[no_mangle]
pub extern "C" fn tts_stop(tts: *mut TTS) -> bool {
pub unsafe extern "C" fn tts_stop(tts: *mut TTS) -> bool {
if tts.is_null() {
return false;
}
match unsafe { tts.as_mut().unwrap().stop() } {
match tts.as_mut().unwrap().stop() {
Ok(_) => true,
Err(e) => {
set_last_error(e.to_string()).unwrap();