Revert "use bufreaders"

This reverts commit f335b99024.

src/cache/fs.rs

@@ -11,7 +11,7 @@ use std::path::{Path, PathBuf};
 use std::pin::Pin;
 use std::task::{Context, Poll};
 use tokio::fs::{create_dir_all, remove_file, File};
-use tokio::io::{AsyncRead, AsyncWriteExt, BufReader, ReadBuf};
+use tokio::io::{AsyncRead, AsyncWriteExt, ReadBuf};
 use tokio::sync::mpsc::UnboundedSender;
 use tokio::sync::watch::{channel, Receiver};
 use tokio::sync::RwLock;
@@ -60,7 +60,7 @@ pub async fn read_file(
             WatchStream::new(status),
         ))
     } else {
-        CacheStream::Completed(FramedRead::new(BufReader::new(file), BytesCodec::new()))
+        CacheStream::Completed(FramedRead::new(file, BytesCodec::new()))
     };
 
     Some(Ok((stream, metadata)))
@@ -154,7 +154,7 @@ pub async fn write_file(
 }
 
 pub struct ConcurrentFsStream {
-    file: Pin<Box<BufReader<File>>>,
+    file: Pin<Box<File>>,
     receiver: Pin<Box<WatchStream<WritingStatus>>>,
     bytes_read: u64,
     bytes_total: Option<NonZeroU64>,
@@ -172,7 +172,7 @@ impl ConcurrentFsStream {
 
     fn from_file(file: File, receiver: WatchStream<WritingStatus>) -> Self {
         Self {
-            file: Box::pin(BufReader::new(file)),
+            file: Box::pin(file),
             receiver: Box::pin(receiver),
             bytes_read: 0,
             bytes_total: None,

src/cache/low_mem.rs

@@ -1,15 +1,13 @@
 //! Low memory caching stuff
 
-use std::path::PathBuf;
 use std::str::FromStr;
-use std::sync::atomic::{AtomicU64, Ordering};
+use std::sync::{atomic::Ordering, Arc};
-use std::sync::Arc;
+use std::{path::PathBuf, sync::atomic::AtomicU64};
 
 use async_trait::async_trait;
 use futures::StreamExt;
 use log::LevelFilter;
-use sqlx::sqlite::SqliteConnectOptions;
+use sqlx::{sqlite::SqliteConnectOptions, ConnectOptions, SqlitePool};
-use sqlx::{ConnectOptions, SqlitePool};
 use tokio::sync::mpsc::{channel, unbounded_channel, Sender, UnboundedSender};
 use tokio_stream::wrappers::ReceiverStream;
 

src/cache/mod.rs

@@ -13,7 +13,6 @@ use futures::{Stream, StreamExt};
 use serde::{Deserialize, Serialize};
 use thiserror::Error;
 use tokio::fs::File;
-use tokio::io::BufReader;
 use tokio_util::codec::{BytesCodec, FramedRead};
 
 pub use fs::UpstreamError;
@@ -171,7 +170,7 @@ pub trait Cache: Send + Sync {
 pub enum CacheStream {
     Concurrent(ConcurrentFsStream),
     Memory(MemStream),
-    Completed(FramedRead<BufReader<File>, BytesCodec>),
+    Completed(FramedRead<File, BytesCodec>),
 }
 
 impl From<CachedImage> for CacheStream {

src/main.rs

@@ -1,4 +1,4 @@
-#![warn(clippy::pedantic, clippy::nursery, clippy::perf)]
+#![warn(clippy::pedantic, clippy::nursery)]
 // We're end users, so these is ok
 #![allow(clippy::module_name_repetitions)]