From f832af5e832a5be2804e5613a73e7cc75428956c Mon Sep 17 00:00:00 2001
From: Mikko Rasa <tdb@tdb.fi>
Date: Sun, 31 Oct 2021 20:04:27 +0200
Subject: [PATCH] Remove the broken IO::Filtered class

---
 source/io/file.h     |  1 -
 source/io/filtered.h | 94 --------------------------------------------
 2 files changed, 95 deletions(-)
 delete mode 100644 source/io/filtered.h

diff --git a/source/io/file.h b/source/io/file.h
index 36f3770..292897b 100644
--- a/source/io/file.h
+++ b/source/io/file.h
@@ -4,7 +4,6 @@
 #include <stdexcept>
 #include <string>
 #include "buffered.h"
-#include "filtered.h"
 #include "handle.h"
 #include "seekable.h"
 
diff --git a/source/io/filtered.h b/source/io/filtered.h
deleted file mode 100644
index 2092743..0000000
--- a/source/io/filtered.h
+++ /dev/null
@@ -1,94 +0,0 @@
-#ifndef MSP_IO_FILTERED_H_
-#define MSP_IO_FILTERED_H_
-
-namespace Msp {
-namespace IO {
-
-/**
-This class is broken by design.  Do not use.  It exposes base class methods in
-an unsafe and misleading way.  In particular, a Filtered<File, Buffered> causes
-seeks to behave incorrectly.
-*/
-template<typename B, typename F>
-class Filtered: public B
-{
-private:
-	struct Activator
-	{
-		Filtered &f;
-
-		Activator(Filtered &f_): f(f_) { f.active = true; }
-		~Activator() { f.active = false; }
-	};
-
-	F filter;
-	bool active;
-
-public:
-	Filtered(): filter(*this), active(false) { }
-	~Filtered() { active = true; }
-
-	template<typename A0>
-	Filtered(A0 a0): B(a0), filter(*this), active(false) { }
-
-	template<typename A0, typename A1>
-	Filtered(A0 a0, A1 a1): B(a0, a1), filter(*this), active(false) { }
-
-protected:
-	virtual unsigned do_write(const char *b, unsigned s)
-	{
-		if(!active)
-		{
-			Activator a(*this);
-			return filter.write(b, s);
-		}
-		else
-			return B::do_write(b, s);
-	}
-
-	virtual unsigned do_read(char *b, unsigned s)
-	{
-		if(!active)
-		{
-			Activator a(*this);
-			return filter.read(b, s);
-		}
-		else
-			return B::do_read(b, s);
-	}
-
-public:
-	virtual unsigned put(char c)
-	{
-		if(active)
-			return B::put(c);
-
-		Activator a(*this);
-		return filter.put(c);
-	}
-
-	virtual bool getline(std::string &l)
-	{
-		if(active)
-			return B::getline(l);
-
-		Activator a(*this);
-		return filter.getline(l);
-	}
-
-	virtual int get()
-	{
-		if(active)
-			return B::get();
-			
-		Activator a(*this);
-		return filter.get();
-	}
-
-	F &get_filter() { return filter; }
-};
-
-} // namespace IO
-} // namespace Msp
-
-#endif
-- 
2.43.0