clippy: Fix missing safety warnings, enable in CI. (#227)

Author: waywardmonkeys

.github/workflows/main.yml

@@ -63,6 +63,9 @@ jobs:
       - name: Cargo build
         run: cargo build --workspace --features bundled
 
+      - name: Cargo clippy
+        run: cargo clippy --workspace --features bundled -- -D warnings
+
       - name: Cargo test
         run: cargo test --workspace --features bundled
         env:

harfbuzz/src/blob.rs

@@ -87,6 +87,10 @@ impl<'a> Blob<'a> {
     }
 
     /// Construct a `Blob` from a raw pointer. Takes ownership of the blob.
+    ///
+    /// # Safety
+    ///
+    /// The pointer must be valid and must not be used after this function is called.
     pub unsafe fn from_raw(raw: *mut sys::hb_blob_t) -> Self {
         Blob {
             raw,

harfbuzz/src/buffer.rs

@@ -83,6 +83,10 @@ impl Buffer {
     }
 
     /// Construct a `Buffer` from a raw pointer. Takes ownership of the buffer.
+    ///
+    /// # Safety
+    ///
+    /// The pointer must be valid and must not be used after this function is called.
     pub unsafe fn from_raw(raw: *mut sys::hb_buffer_t) -> Self {
         Buffer { raw }
     }

harfbuzz/src/language.rs

@@ -39,6 +39,11 @@ impl Language {
             .unwrap()
     }
 
+    /// Construct a `Language` from a raw pointer.
+    ///
+    /// # Safety
+    ///
+    /// The pointer must be a valid pointer.
     pub unsafe fn from_raw(raw: sys::hb_language_t) -> Self {
         Language { raw }
     }