Removed MemoryOutStream, MemoryInStream

BinaryWriter/BinaryReader now implements memory buffer functionality which previously needed two chained instances of a Buffer/Reader. This was originally expected to be an abstraction for file and other stream access but this is not going to be useful so may as well collapse the functionality.

This also eliminates the need for stack-aware ref-counting which is the real reason for wanting to get rid of this code. This was a very old experimental feature which turned out to be a bad idea.

This also removes the /cas/batch endpoint

1 files changed, 4 insertions, 6 deletions

diff --git a/zenserver/projectstore.cpp b/zenserver/projectstore.cpp
index 1f4239b23..1a9eb2c67 100644
--- a/zenserver/projectstore.cpp
+++ b/zenserver/projectstore.cpp
@@ -635,8 +635,7 @@ ProjectStore::Project::Read()
 void
 ProjectStore::Project::Write()
 {
-	MemoryOutStream Mem;
-	BinaryWriter	Writer(Mem);
+	BinaryWriter Mem;
 
 	CbObjectWriter Cfg;
 	Cfg << "id" << Identifier;
@@ -644,7 +643,7 @@ ProjectStore::Project::Write()
 	Cfg << "project" << ProjectRootDir;
 	Cfg << "engine" << EngineRootDir;
 
-	Cfg.Save(Writer);
+	Cfg.Save(Mem);
 
 	CreateDirectories(m_OplogStoragePath);
 
@@ -951,9 +950,8 @@ HttpProjectService::HttpProjectService(CasStore& Store, ProjectStore* Projects)
 
 			// Parse Request
 
-			IoBuffer	   Payload = HttpReq.ReadPayload();
-			MemoryInStream MemIn(Payload.Data(), Payload.Size());
-			BinaryReader   Reader(MemIn);
+			IoBuffer	 Payload = HttpReq.ReadPayload();
+			BinaryReader Reader(Payload);
 
 			struct RequestHeader
 			{