From 058625b24919e6d9957f0e3a3def06fae2550c94 Mon Sep 17 00:00:00 2001
From: Paul Delafosse <paul.delafosse@protonmail.com>
Date: Fri, 13 May 2022 08:36:58 +0200
Subject: [PATCH] chore: remove dummy benchmark

---
 benches/simple_lookup.rs | 24 +++++++++---------------
 1 file changed, 9 insertions(+), 15 deletions(-)

diff --git a/benches/simple_lookup.rs b/benches/simple_lookup.rs
index c5adf4f..8374103 100644
--- a/benches/simple_lookup.rs
+++ b/benches/simple_lookup.rs
@@ -2,12 +2,6 @@ use criterion::{criterion_group, criterion_main, BenchmarkId, Criterion};
 use freedesktop_icons::lookup;
 use gtk4::{IconLookupFlags, IconTheme, TextDirection};
 
-pub fn simple_bench(c: &mut Criterion) {
-    c.bench_function("firefox lookup", |b| {
-        b.iter(|| lookup("firefox").with_theme("Papirus").with_cache().find())
-    });
-}
-
 pub fn bench_lookups(c: &mut Criterion) {
     let mut group = c.benchmark_group("ComparisonsLookups");
 
@@ -15,28 +9,28 @@ pub fn bench_lookups(c: &mut Criterion) {
         "user-home", // (Best case) An icon that can be found in the current theme
         "video-single-display-symbolic", // An icon that can be found in the parent theme
         "firefox",   // An icon that can be found in the hicolor default theme
-        "archlinux-logo", // An icon that resides in /usr/share/pixmap
+        "archlinux-logo", // An icon that resides in /usr/share/pixmaps
         "not-found", // (Worst case) An icon that does not exist
     ];
 
     for arg in args {
+        group.bench_with_input(BenchmarkId::new("freedesktop-icons", arg), arg, |b, arg| {
+            b.iter(|| lookup(arg).with_theme("Arc").find());
+        });
+
         group.bench_with_input(
-            BenchmarkId::new("freedesktop-icons lookup", arg),
+            BenchmarkId::new("freedesktop-icons-cache", arg),
             arg,
             |b, arg| {
-                b.iter(|| lookup(arg).with_theme("Arc").find());
+                b.iter(|| lookup(arg).with_theme("Arc").with_cache().find());
             },
         );
 
-        group.bench_with_input(BenchmarkId::new("lookup with cache", arg), arg, |b, arg| {
-            b.iter(|| lookup(arg).with_theme("Arc").with_cache().find());
-        });
-
-        group.bench_with_input(BenchmarkId::new("lookup linicon", arg), arg, |b, arg| {
+        group.bench_with_input(BenchmarkId::new("linicon", arg), arg, |b, arg| {
             b.iter(|| linicon::lookup_icon(arg).from_theme("Arc").next());
         });
 
-        group.bench_with_input(BenchmarkId::new("lookup gtk", arg), arg, |b, arg| {
+        group.bench_with_input(BenchmarkId::new("gtk", arg), arg, |b, arg| {
             gtk4::init().unwrap();
             let theme = IconTheme::new();
             b.iter(|| {