doc: include arg -DUSE_EXTERNAL_DEFAULT_CALLBACKS=1 for cmake
What the developer wrote
Authored by kevkevinpal
doc: include arg -DUSE_EXTERNAL_DEFAULT_CALLBACKS=1 for cmake
What changed, and why it matters
This is a one-line documentation update in a header file comment. It adds the CMake equivalent of an existing Autotools build option so developers know how to enable external default callbacks when building with CMake. No code behavior changes.
No security action required. This is a documentation-only change.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The commit modifies a comment in include/secp256k1.h to document that USE_EXTERNAL_DEFAULT_CALLBACKS can be enabled via -DSECP256K1_USE_EXTERNAL_DEFAULT_CALLBACKS=ON in CMake builds, in addition to the existing –enable-external-default-callbacks for GNU Autotools. The diff contains no functional changes, no macro renames, no build system edits, and no API changes.
Changed components
include/secp256k1.h (documentation comment only)Inspect captured patch +2 / −1
diff --git a/include/secp256k1.h b/include/secp256k1.h
index b0a13b1..2799d4a 100644
--- a/include/secp256k1.h
+++ b/include/secp256k1.h
@@ -349,7 +349,8 @@ SECP256K1_API void secp256k1_context_destroy(
* writes the message to stderr and calls abort. This default callback can be
* replaced at link time if the preprocessor macro
* USE_EXTERNAL_DEFAULT_CALLBACKS is defined, which is the case if the build
- * has been configured with --enable-external-default-callbacks. Then the
+ * has been configured with --enable-external-default-callbacks (GNU Autotools) or
+ * -DSECP256K1_USE_EXTERNAL_DEFAULT_CALLBACKS=ON (CMake). Then the
* following two symbols must be provided to link against:
* - void secp256k1_default_illegal_callback_fn(const char *message, void *data);
* - void secp256k1_default_error_callback_fn(const char *message, void *data);
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.