NodeTask considered 'must_use' (#92)

lemunozm · Jun 3, 2021 · 73504d4 · 73504d4
1 parent 54fe31b
commit 73504d4
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,8 @@
 # Changelog
 
+## Release 0.14.1
+- `NodeTask` is now considered *must_use*.
+
 ## Release 0.14.0
 - Asynchronous connections: `NetworkController::connect()` behaviour modified.
 Now it performs a non-blocking connection. Previous behaviour with `connect_sync` version.

diff --git a/Cargo.lock b/Cargo.lock
diff --git a/Cargo.toml b/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "message-io"
-version = "0.14.0"
+version = "0.14.1"
 authors = ["lemunozm <[email protected]>"]
 edition = "2018"
 readme = "README.md"

diff --git a/src/node.rs b/src/node.rs
@@ -496,6 +496,7 @@ impl<S: Send + 'static> Drop for NodeListener<S> {
 /// The destruction of this entity will block until the task is finished.
 /// If you want to "unblock" the thread that drops this entity call to
 /// [`NodeHandler::stop()`] before or from another thread.
+#[must_use = "The NodeTask must be used or the asynchronous task will be dropped in return"]
 pub struct NodeTask {
     network_thread: NamespacedThread<()>,
     signal_thread: NamespacedThread<()>,