Add a 'dont_replace_c_symbols' feature for not replacing C's symbols with rust. isn't compatible with no-std

Author: elichai

Cargo.toml

@@ -33,6 +33,7 @@ std = ["rand/std"]
 recovery = []
 endomorphism = []
 lowmemory = []
+dont_replace_c_symbols = []
 
 [dev-dependencies]
 rand = "0.6"

build.rs

@@ -52,14 +52,15 @@ fn main() {
                .define("USE_NUM_NONE", Some("1"))
                .define("USE_FIELD_INV_BUILTIN", Some("1"))
                .define("USE_SCALAR_INV_BUILTIN", Some("1"))
-               .define("ENABLE_MODULE_ECDH", Some("1"))
-               .define("USE_EXTERNAL_DEFAULT_CALLBACKS", Some("1"));
+               .define("ENABLE_MODULE_ECDH", Some("1"));
 
     if cfg!(feature = "lowmemory") {
         base_config.define("ECMULT_WINDOW_SIZE", Some("4")); // A low-enough value to consume neglible memory
     } 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"));
     #[cfg(feature = "recovery")]

src/ffi.rs

@@ -256,7 +256,7 @@ extern "C" {
 }
 
 
-#[cfg(feature = "std")]
+#[cfg(all(feature = "std", not(feature = "dont_replace_c_symbols")))]
 #[no_mangle]
 /// A reimplementation of the C function `secp256k1_context_create` in rust.
 ///
@@ -281,7 +281,7 @@ pub unsafe extern "C" fn secp256k1_context_create(flags: c_uint) -> *mut Context
     secp256k1_context_preallocated_create(ptr as *mut c_void, flags)
 }
 
-#[cfg(feature = "std")]
+#[cfg(all(feature = "std", not(feature = "dont_replace_c_symbols")))]
 #[no_mangle]
 /// A reimplementation of the C function `secp256k1_context_destroy` in rust.
 ///
@@ -300,6 +300,7 @@ pub unsafe extern "C" fn secp256k1_context_destroy(ctx: *mut Context) {
 }
 
 
+#[cfg(not(feature = "dont_replace_c_symbols"))]
 #[no_mangle]
 /// **This function is an override for the C function, this is the an edited version of the original description:**
 ///
@@ -326,6 +327,7 @@ pub unsafe extern "C" fn secp256k1_default_illegal_callback_fn(message: *const c
     panic!("[libsecp256k1] illegal argument. {}", msg);
 }
 
+#[cfg(not(feature = "dont_replace_c_symbols"))]
 #[no_mangle]
 /// **This function is an override for the C function, this is the an edited version of the original description:**
 ///

src/lib.rs

@@ -717,6 +717,7 @@ 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) };