Remove dependency on once_cell.

Cargo.toml

@@ -7,7 +7,7 @@ keywords = ["web", "http", "server", "async", "threaded"]
 license = "MIT OR Apache-2.0"
 name = "servlin"
 repository = "https://github.com/mleonhard/servlin"
-version = "0.5.0"
+version = "0.5.1"
 
 [features]
 default = []
@@ -22,7 +22,6 @@ futures-io = "0.3"
 futures-lite = "2"
 include_dir = { version = "0.7", optional = true }
 #libflate = "1"
-once_cell = "1"
 permit = "^0.2"
 rand = { version = "0.8", features = ["small_rng"] }
 safe-regex = "0.3"

Readme.md

@@ -105,7 +105,7 @@ Symbols:
 
 Functions  Expressions  Impls  Traits  Methods  Dependency
 
-0/0        0/0          0/0    0/0     0/0      🔒  servlin 0.5.0
+0/0        0/0          0/0    0/0     0/0      🔒  servlin 0.5.1
 0/0        0/0          0/0    0/0     0/0      🔒  ├── async-fs 2.1.2
 4/4        222/222      40/40  0/0     13/13    ☢️  │   ├── async-lock 3.4.0
 0/0        2/2          0/0    0/0     0/0      ☢️  │   │   ├── event-listener-strategy 0.5.2
@@ -165,7 +165,6 @@ Functions  Expressions  Impls  Traits  Methods  Dependency
 0/0        0/0          0/0    0/0     0/0      ❓  │   └── futures-io 0.3.31
 0/0        0/0          0/0    0/0     0/0      ❓  ├── futures-io 0.3.31
 0/0        0/0          0/0    0/0     0/0      ❓  ├── futures-lite 2.3.0
-0/0        74/117       5/9    0/0     2/4      ☢️  ├── once_cell 1.20.2
 0/0        0/0          0/0    0/0     0/0      🔒  ├── permit 0.2.1
 0/0        32/32        0/0    0/0     0/0      ☢️  ├── rand 0.8.5
 1/90       10/596       0/2    0/0     5/69     ☢️  │   ├── libc 0.2.161
@@ -220,6 +219,7 @@ Functions  Expressions  Impls  Traits  Methods  Dependency
 See [rust-webserver-comparison.md](https://github.com/mleonhard/servlin/blob/main/rust-webserver-comparison.md).
 
 # Changelog
+- v0.5.1 2024-10-26 - Remove dependency on `once_cell`.
 - v0.5.0 2024-10-21 - Remove `LogFileWriterBuilder`.
 - v0.4.3 - Implement `From<Cow<'_, str>>` and `From<&Path>` for `TagValue`.
 - v0.4.2 - Implement `Seek` for `BodyReader`.

check-readme.sh

@@ -5,7 +5,7 @@ cat Readme.md |perl -0777 -pe 's/(# Cargo Geiger Safety Report).+?```.+?```/$1/s
 cargo readme --no-title --no-indent-headings >Readme.md.tmp
 diff Readme.md.pruned Readme.md.tmp || (
   set +x
-  echo "ERROR: Readme.md is stale" >&2
+  echo "ERROR: $(pwd)/Readme.md is stale" >&2
   exit 1
 )
 rm -f Readme.md.pruned Readme.md.tmp

check.sh

@@ -3,8 +3,8 @@ set -e
 set -x
 time cargo check --all-targets --all-features
 time cargo build --all-targets --all-features
-time cargo fmt -- --check
-time cargo clippy --all-targets --all-features -- -D clippy::pedantic
+time cargo fmt --all
+time cargo clippy --all-targets --all-features --allow-dirty --allow-staged --fix -- -D clippy::pedantic
 time cargo test --all-targets --all-features
 time cargo test --doc --all-features
 ./check-readme.sh

git-push.sh

@@ -0,0 +1,5 @@
+#!/usr/bin/env sh
+set -e
+set -x
+../check-all.sh "$@"
+git push --follow-tags

src/lib.rs

@@ -103,6 +103,7 @@
 //! See [rust-webserver-comparison.md](https://github.com/mleonhard/servlin/blob/main/rust-webserver-comparison.md).
 //!
 //! # Changelog
+//! - v0.5.1 2024-10-26 - Remove dependency on `once_cell`.
 //! - v0.5.0 2024-10-21 - Remove `LogFileWriterBuilder`.
 //! - v0.4.3 - Implement `From<Cow<'_, str>>` and `From<&Path>` for `TagValue`.
 //! - v0.4.2 - Implement `Seek` for `BodyReader`.

src/log/logger.rs

@@ -96,8 +96,7 @@ impl GlobalLoggerState {
     }
 }
 
-pub static GLOBAL_LOGGER: once_cell::sync::OnceCell<Mutex<GlobalLoggerState>> =
-    once_cell::sync::OnceCell::new();
+pub static GLOBAL_LOGGER: Mutex<GlobalLoggerState> = Mutex::new(GlobalLoggerState::None);
 
 thread_local! {
     pub static THREAD_LOCAL_TAGS: RefCell<Vec<Tag>> = const { RefCell::new(Vec::new()) };
@@ -109,7 +108,6 @@ pub struct GlobalLoggerAlreadySetError {}
 #[allow(clippy::module_name_repetitions)]
 pub fn lock_global_logger() -> MutexGuard<'static, GlobalLoggerState> {
     GLOBAL_LOGGER
-        .get_or_init(|| Mutex::new(GlobalLoggerState::None))
         .lock()
         .unwrap_or_else(std::sync::PoisonError::into_inner)
 }