From 3005c54a7e5d232067cc9e9f2b72be7e9a0a3df8 Mon Sep 17 00:00:00 2001 From: Ivan Folgueira Bande Date: Tue, 29 Jul 2025 09:28:39 +0200 Subject: [PATCH] adjust as per Franck's comments --- FURPS/application/nim_ffi.md | 14 +++++++------- draft-roadmap/nim_usage_improvements.md | 2 +- 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/FURPS/application/nim_ffi.md b/FURPS/application/nim_ffi.md index c030e08..7c8656d 100644 --- a/FURPS/application/nim_ffi.md +++ b/FURPS/application/nim_ffi.md @@ -3,13 +3,14 @@ ## Functionality 1. Provides the core logic needed to expose any synchronous or asynchronous Nim library to a C-FFI library. -2. The exposed C library can be used in Golang. -3. The exposed C library can be used in Rust. -4. The exposed C library can be used in Python. ## Usability 1. Introduce new pragma definitions, such as `{.ffi.}`, to appropriately annotate types and procedures. +2. Any Nim project can use it and can be installed using Nimble, +similarly to how nim-chronos is imported. +3. The interaction with the exposed C library is through JSON. +4. The interaction with the exposed C library is through protobuf. ## Reliability @@ -22,10 +23,9 @@ ## Supportability -1. Any Nim project can use it and can be installed using Nimble, -similarly to how nim-chronos is imported. -2. The interaction with the exposed C library is through JSON. -3. The interaction with the exposed C library is through protobuf. +1. The exposed C library can be used in Golang. +2. The exposed C library can be used in Rust. +3. The exposed C library can be used in Python. ## + (Privacy, Anonymity, Deployments) diff --git a/draft-roadmap/nim_usage_improvements.md b/draft-roadmap/nim_usage_improvements.md index 73dd00c..1f23aab 100644 --- a/draft-roadmap/nim_usage_improvements.md +++ b/draft-roadmap/nim_usage_improvements.md @@ -85,7 +85,7 @@ This deliverable takes care of the following [Nim FFI](/FURPS/application/nim_ff - F1. - U1. -- S1. +- U2. - +1. `nwaku` repository uses `nim-ffi` to expose the existing `libwaku` functionality. **Checklist**: