Fix clippy warnings

Implement `Eq` for the `Error` type and add safety
comments where missing, preceding unsafe blocks.

Signed-off-by: Alexandra Iordache <[email protected]>

fsjksk

Author: Alexandra Iordache

CHANGELOG.md

@@ -1,3 +1,9 @@
+# [Unreleased]
+
+## Added
+
+- The `Error` type now implements `Eq`.
+
 # v0.2.1
 
 ## Changed

src/lib.rs

@@ -32,7 +32,7 @@ mod endpoint;
 pub use endpoint::RemoteEndpoint;
 
 /// Error conditions that may appear during `EventManager` related operations.
-#[derive(Debug, PartialEq)]
+#[derive(Debug, Eq, PartialEq)]
 pub enum Error {
     #[cfg(feature = "remote_endpoint")]
     /// Cannot send message on channel.

tests/negative_tests.rs

@@ -63,6 +63,7 @@ fn test_handling_errors_in_subscriber() {
     let subscriber = Arc::new(UnixStreamSubscriber::new(sock1));
     event_manager.add_subscriber(subscriber.clone());
 
+    // SAFETY: safe because `sock2` is a valid Unix socket, as asserted by the `unwrap` above.
     unsafe { libc::close(sock2.as_raw_fd()) };
 
     event_manager.run_with_timeout(100).unwrap();
@@ -79,6 +80,7 @@ fn test_handling_errors_in_subscriber() {
         Arc::new(UnixStreamSubscriber::new_with_unregister_on_err(sock1));
     event_manager.add_subscriber(subscriber_with_unregister);
 
+    // SAFETY: safe because `sock2` is a valid Unix socket, as asserted by the `unwrap` above.
     unsafe { libc::close(sock2.as_raw_fd()) };
 
     let ready_list_len = event_manager.run_with_timeout(100).unwrap();