remove SimpleBackend

Author: Robbepop

benches/bench.rs

@@ -5,7 +5,6 @@ use self::setup::{
     BackendBenchmark,
     BenchBucket,
     BenchBuffer,
-    BenchSimple,
     BenchString,
     BENCH_LEN_STRINGS,
     BENCH_STRING_LEN,
@@ -76,7 +75,6 @@ fn bench_get_or_intern_static(c: &mut Criterion) {
             },
         );
     }
-    bench_for_backend::<BenchSimple>(&mut g);
     bench_for_backend::<BenchBucket>(&mut g);
     bench_for_backend::<BenchString>(&mut g);
     bench_for_backend::<BenchBuffer>(&mut g);
@@ -103,7 +101,6 @@ fn bench_get_or_intern_fill_with_capacity(c: &mut Criterion) {
             },
         );
     }
-    bench_for_backend::<BenchSimple>(&mut g);
     bench_for_backend::<BenchBucket>(&mut g);
     bench_for_backend::<BenchString>(&mut g);
     bench_for_backend::<BenchBuffer>(&mut g);
@@ -130,7 +127,6 @@ fn bench_get_or_intern_fill(c: &mut Criterion) {
             },
         );
     }
-    bench_for_backend::<BenchSimple>(&mut g);
     bench_for_backend::<BenchBucket>(&mut g);
     bench_for_backend::<BenchString>(&mut g);
     bench_for_backend::<BenchBuffer>(&mut g);
@@ -157,7 +153,6 @@ fn bench_get_or_intern_already_filled(c: &mut Criterion) {
             },
         );
     }
-    bench_for_backend::<BenchSimple>(&mut g);
     bench_for_backend::<BenchBucket>(&mut g);
     bench_for_backend::<BenchString>(&mut g);
     bench_for_backend::<BenchBuffer>(&mut g);
@@ -184,7 +179,6 @@ fn bench_resolve_already_filled(c: &mut Criterion) {
             },
         );
     }
-    bench_for_backend::<BenchSimple>(&mut g);
     bench_for_backend::<BenchBucket>(&mut g);
     bench_for_backend::<BenchString>(&mut g);
     bench_for_backend::<BenchBuffer>(&mut g);
@@ -211,7 +205,6 @@ fn bench_get_already_filled(c: &mut Criterion) {
             },
         );
     }
-    bench_for_backend::<BenchSimple>(&mut g);
     bench_for_backend::<BenchBucket>(&mut g);
     bench_for_backend::<BenchString>(&mut g);
     bench_for_backend::<BenchBuffer>(&mut g);
@@ -246,7 +239,6 @@ fn bench_iter_already_filled(c: &mut Criterion) {
             },
         );
     }
-    bench_for_backend::<BenchSimple>(&mut g);
     bench_for_backend::<BenchBucket>(&mut g);
     bench_for_backend::<BenchString>(&mut g);
     bench_for_backend::<BenchBuffer>(&mut g);

benches/setup.rs

@@ -1,5 +1,5 @@
 use string_interner::{
-    backend::{Backend, BucketBackend, BufferBackend, SimpleBackend, StringBackend},
+    backend::{Backend, BucketBackend, BufferBackend, StringBackend},
     DefaultSymbol,
     StringInterner,
 };
@@ -118,12 +118,6 @@ impl BackendBenchmark for BenchBucket {
     type Backend = BucketBackend<DefaultSymbol>;
 }
 
-pub struct BenchSimple;
-impl BackendBenchmark for BenchSimple {
-    const NAME: &'static str = "SimpleBackend";
-    type Backend = SimpleBackend<DefaultSymbol>;
-}
-
 pub struct BenchString;
 impl BackendBenchmark for BenchString {
     const NAME: &'static str = "StringBackend";

src/backend/mod.rs

@@ -6,16 +6,10 @@
 
 mod bucket;
 mod buffer;
-mod simple;
 mod string;
 
 #[cfg(feature = "backends")]
-pub use self::{
-    bucket::BucketBackend,
-    buffer::BufferBackend,
-    simple::SimpleBackend,
-    string::StringBackend,
-};
+pub use self::{bucket::BucketBackend, buffer::BufferBackend, string::StringBackend};
 use crate::Symbol;
 
 /// The default backend recommended for general use.

src/backend/simple.rs

@@ -118,10 +118,6 @@
 //! Some operations are even slightly more efficient and it consumes less memory.
 //! However, all this is at the costs of a less efficient resolution of symbols.
 //! Note that the symbols generated by the `BufferBackend` are not contiguous.
-//!
-//! ### Simple Backend
-//!
-//! Never use this one for real use cases!
 
 #[cfg(not(feature = "std"))]
 extern crate alloc;

src/lib.rs

@@ -41,14 +41,6 @@ impl BackendStats for backend::BucketBackend<DefaultSymbol> {
     const NAME: &'static str = "BucketBackend";
 }
 
-impl BackendStats for backend::SimpleBackend<DefaultSymbol> {
-    const MIN_OVERHEAD: f64 = 2.1;
-    const MAX_OVERHEAD: f64 = 2.33;
-    const MAX_ALLOCATIONS: usize = 1000040;
-    const MAX_DEALLOCATIONS: usize = 38;
-    const NAME: &'static str = "SimpleBackend";
-}
-
 impl BackendStats for backend::StringBackend<DefaultSymbol> {
     const MIN_OVERHEAD: f64 = 1.7;
     const MAX_OVERHEAD: f64 = 1.93;
@@ -377,12 +369,6 @@ mod bucket_backend {
     gen_tests_for_backend!(backend::BucketBackend<DefaultSymbol>);
 }
 
-mod simple_backend {
-    use super::*;
-
-    gen_tests_for_backend!(backend::SimpleBackend<DefaultSymbol>);
-}
-
 mod string_backend {
     use super::*;