Merge pull request #5 from notdijon/tweak

Tweak

Author: notdijon

.cargo/config

@@ -1,6 +1,6 @@
 [package]
 name = "hexavalent"
-version = "0.1.5"
+version = "0.1.6"
 authors = ["notdijon <[email protected]>"]
 license = "MIT"
 description = "Write HexChat plugins in Rust."

Cargo.toml

@@ -96,7 +96,7 @@ pub(crate) unsafe fn word_to_iter<'a>(
             while n > 0 {
                 let elem = unsafe { *self.word };
                 if elem.is_null() {
-                    // nothing
+                    break;
                 } else {
                     // Safety: elem is not null, so there is at least one more element in the array (possibly null)
                     self.word = unsafe { self.word.add(1) };

src/ffi.rs

@@ -29,6 +29,7 @@ pub(crate) mod private {
         const NAME: *const c_char;
     }
 
+    #[allow(unreachable_pub)]
     pub trait FromInfoValue: Sized {
         fn from_info_value(info: Option<&str>) -> Self;
     }

src/info.rs

@@ -90,6 +90,7 @@
 //! In debug mode (specifically, when `debug_assertions` is enabled), the current thread ID is checked every time the plugin is invoked,
 //! which can help detect misbehavior.
 
+#![allow(clippy::result_unit_err)]
 #![warn(
     missing_debug_implementations,
     missing_docs,

src/lib.rs

@@ -33,6 +33,7 @@ pub(crate) mod private {
         const NAME: *const c_char;
     }
 
+    #[allow(unreachable_pub)]
     pub trait FromListElem: Sized {
         fn from_list_elem(elem: ListElem<'_>) -> Self;
     }

src/list.rs

@@ -31,13 +31,15 @@ pub(crate) mod private {
         const NAME: *const c_char;
     }
 
+    #[allow(unreachable_pub)]
     #[derive(Debug)]
     pub enum PrefValue<'a> {
         Str(&'a str),
         Int(i32),
         Bool(bool),
     }
 
+    #[allow(unreachable_pub)]
     pub trait FromPrefValue: Sized {
         fn from_pref_value(pref: PrefValue<'_>) -> Result<Self, ()>;
     }