Merge pull request #83 from elast0ny/fix/logging

Fix logging feature issues

Author: elast0ny

Cargo.toml

@@ -1,7 +1,7 @@
 [package]
 name = "shared_memory"
 description = "A user friendly crate that allows you to share memory between processes"
-version = "0.12.2"
+version = "0.12.3"
 authors = ["ElasT0ny <elast0ny00@gmail.com>"]
 license = "MIT OR Apache-2.0"
 edition = "2018"
@@ -22,7 +22,8 @@ categories = [
 exclude = ["ci/*", ".github/*"]
 
 [features]
-default = ["log/release_max_level_off"]
+default = []
+logging = ["log"]
 
 [dependencies]
 cfg-if = "1.0"

src/lib.rs

@@ -9,7 +9,21 @@ use std::fs::remove_file;
 use std::path::{Path, PathBuf};
 
 use cfg_if::cfg_if;
-use log::*;
+
+#[cfg(feature = "logging")]
+pub use log;
+#[cfg(not(feature = "logging"))]
+#[allow(unused_macros)]
+mod log {
+    macro_rules! trace (($($tt:tt)*) => {{}});
+    macro_rules! debug (($($tt:tt)*) => {{}});
+    macro_rules! info (($($tt:tt)*) => {{}});
+    macro_rules! warn (($($tt:tt)*) => {{}});
+    macro_rules! error (($($tt:tt)*) => {{}});
+    pub(crate) use {debug, trace};
+}
+
+use crate::log::*;
 
 mod error;
 pub use error::*;
@@ -27,7 +41,7 @@ cfg_if! {
     }
 }
 
-#[derive(Clone)]
+#[derive(Clone, Default)]
 /// Struct used to configure different parameters before creating a shared memory mapping
 pub struct ShmemConf {
     owner: bool,
@@ -47,17 +61,7 @@ impl Drop for ShmemConf {
         }
     }
 }
-impl Default for ShmemConf {
-    fn default() -> Self {
-        Self {
-            owner: false,
-            os_id: None,
-            overwrite_flink: false,
-            flink_path: None,
-            size: 0,
-        }
-    }
-}
+
 impl ShmemConf {
     /// Create a new default shmem config
     pub fn new() -> Self {

src/unix.rs

@@ -1,7 +1,7 @@
 use std::os::unix::io::RawFd;
 use std::ptr::null_mut;
 
-use log::*;
+use crate::log::*;
 use nix::fcntl::OFlag;
 use nix::sys::mman::{mmap, munmap, shm_open, shm_unlink, MapFlags, ProtFlags};
 use nix::sys::stat::{fstat, Mode};

src/windows.rs

@@ -4,7 +4,7 @@ use std::os::windows::{fs::OpenOptionsExt, io::AsRawHandle};
 use std::path::PathBuf;
 use std::ptr::null_mut;
 
-use log::*;
+use crate::log::*;
 use win_sys::*;
 
 use crate::ShmemError;