From 889e9a8740d6d8873a8e8222276110cf052075e5 Mon Sep 17 00:00:00 2001 From: Elichai Turkel Date: Sun, 9 Feb 2020 16:37:39 +0200 Subject: [PATCH 1/2] Fix wrong feature name external-symbols --- secp256k1-sys/src/lib.rs | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/secp256k1-sys/src/lib.rs b/secp256k1-sys/src/lib.rs index 7844464..df0c3b5 100644 --- a/secp256k1-sys/src/lib.rs +++ b/secp256k1-sys/src/lib.rs @@ -295,7 +295,7 @@ extern "C" { // Returns: a newly created context object. // In: flags: which parts of the context to initialize. #[no_mangle] -#[cfg(all(feature = "std", not(feature = "external_symbols")))] +#[cfg(all(feature = "std", not(feature = "external-symbols")))] pub unsafe extern "C" fn rustsecp256k1_v0_1_1_context_create(flags: c_uint) -> *mut Context { use std::mem; assert!(mem::align_of::() >= mem::align_of::()); @@ -312,7 +312,7 @@ pub unsafe extern "C" fn rustsecp256k1_v0_1_1_context_create(flags: c_uint) -> * secp256k1_context_preallocated_create(ptr as *mut c_void, flags) } -#[cfg(all(feature = "std", not(feature = "external_symbols")))] +#[cfg(all(feature = "std", not(feature = "external-symbols")))] pub unsafe fn secp256k1_context_create(flags: c_uint) -> *mut Context { rustsecp256k1_v0_1_1_context_create(flags) } @@ -324,7 +324,7 @@ pub unsafe fn secp256k1_context_create(flags: c_uint) -> *mut Context { /// The pointer shouldn't be used after passing to this function, consider it as passing it to `free()`. /// #[no_mangle] -#[cfg(all(feature = "std", not(feature = "external_symbols")))] +#[cfg(all(feature = "std", not(feature = "external-symbols")))] pub unsafe extern "C" fn rustsecp256k1_v0_1_1_context_destroy(ctx: *mut Context) { secp256k1_context_preallocated_destroy(ctx); let ctx: *mut usize = ctx as *mut usize; @@ -335,7 +335,7 @@ pub unsafe extern "C" fn rustsecp256k1_v0_1_1_context_destroy(ctx: *mut Context) let _ = Box::from_raw(slice as *mut [usize]); } -#[cfg(all(feature = "std", not(feature = "external_symbols")))] +#[cfg(all(feature = "std", not(feature = "external-symbols")))] pub unsafe fn secp256k1_context_destroy(ctx: *mut Context) { rustsecp256k1_v0_1_1_context_destroy(ctx) } @@ -360,7 +360,7 @@ pub unsafe fn secp256k1_context_destroy(ctx: *mut Context) { /// See also secp256k1_default_error_callback_fn. /// #[no_mangle] -#[cfg(not(feature = "external_symbols"))] +#[cfg(not(feature = "external-symbols"))] pub unsafe extern "C" fn rustsecp256k1_v0_1_1_default_illegal_callback_fn(message: *const c_char, _data: *mut c_void) { use core::str; let msg_slice = slice::from_raw_parts(message as *const u8, strlen(message)); @@ -383,7 +383,7 @@ pub unsafe extern "C" fn rustsecp256k1_v0_1_1_default_illegal_callback_fn(messag /// See also secp256k1_default_illegal_callback_fn. /// #[no_mangle] -#[cfg(not(feature = "external_symbols"))] +#[cfg(not(feature = "external-symbols"))] pub unsafe extern "C" fn rustsecp256k1_v0_1_1_default_error_callback_fn(message: *const c_char, _data: *mut c_void) { use core::str; let msg_slice = slice::from_raw_parts(message as *const u8, strlen(message)); From 2f897d8a934c42a52a768a1cce58a45cca2ce8fa Mon Sep 17 00:00:00 2001 From: Elichai Turkel Date: Sun, 9 Feb 2020 16:37:59 +0200 Subject: [PATCH 2/2] Removed no longer used dont_replace_c_symbols feature --- secp256k1-sys/build.rs | 1 - src/lib.rs | 1 - 2 files changed, 2 deletions(-) diff --git a/secp256k1-sys/build.rs b/secp256k1-sys/build.rs index 0fb1173..af91b5a 100644 --- a/secp256k1-sys/build.rs +++ b/secp256k1-sys/build.rs @@ -63,7 +63,6 @@ fn main() { } else { base_config.define("ECMULT_WINDOW_SIZE", Some("15")); // This is the default in the configure file (`auto`) } - #[cfg(not(feature = "dont_replace_c_symbols"))] base_config.define("USE_EXTERNAL_DEFAULT_CALLBACKS", Some("1")); #[cfg(feature = "endomorphism")] base_config.define("USE_ENDOMORPHISM", Some("1")); diff --git a/src/lib.rs b/src/lib.rs index 8b61372..4148827 100644 --- a/src/lib.rs +++ b/src/lib.rs @@ -734,7 +734,6 @@ mod tests { #[test] - #[cfg(not(feature = "dont_replace_c_symbols"))] fn test_manual_create_destroy() { let ctx_full = unsafe { ffi::secp256k1_context_create(AllPreallocated::FLAGS) }; let ctx_sign = unsafe { ffi::secp256k1_context_create(SignOnlyPreallocated::FLAGS) };