- use the lump cache directly for FileData.

This was the final puzzle piece to allow using memory mapped files. :)

src/common/filesystem/filesystem.cpp

@@ -1518,29 +1518,6 @@ bool FileSystem::CreatePathlessCopy(const char *name, int id, int /*flags*/)
 	return true;
 }
 
-// FileData -----------------------------------------------------------------
-
-FileData::FileData (const FileData &copy)
-{
-	Block = copy.Block;
-}
-
-FileData &FileData::operator = (const FileData &copy)
-{
-	Block = copy.Block;
-	return *this;
-}
-
-FileData::FileData (FResourceLump* lump)
-{
-	auto size = lump->LumpSize;
-	Block.resize(1 + size);
-	memcpy(Block.data(), lump->Lock(), size);
-	Block[size] = 0;
-	lump->Unlock();
-}
-
-
 //==========================================================================
 //
 // PrintLastError

src/common/filesystem/filesystem.h

@@ -28,18 +28,34 @@ union LumpShortName
 class FileData
 {
 public:
-	FileData () = default;
+	FileData() { lump = nullptr; }
-	FileData (const FileData &copy);
+	const void *GetMem () { return lump->Cache; }
-	FileData &operator= (const FileData &copy);
+	size_t GetSize () { return lump->LumpSize; }
-	const void *GetMem () { return Block.size() <= 1 ? NULL : (void *)Block.data(); }
+	const char* GetString () const { return (const char*)lump->Cache; }
-	size_t GetSize () { return Block.size(); }
+	const uint8_t* GetBytes() const { return (const uint8_t*)lump->Cache; }
-	const char* GetString () const { return (const char*)Block.data(); }
+
-	const uint8_t* GetBytes() const { return Block.data(); }
+	FileData& operator = (const FileData& copy) = delete;
+
+	FileData(const FileData& copy)
+	{
+		lump = copy.lump;
+		lump->Lock();
+	}
+
+	~FileData()
+	{
+		if (lump) lump->Unlock();
+	}
+
 
 private:
-	FileData (FResourceLump* lump);
+	FileData(FResourceLump* nlump)
+	{
+		lump = nlump;
+		if (lump) lump->Lock();
+	}
 
-	std::vector<uint8_t> Block;
+	FResourceLump* lump;
 
 	friend class FileSystem;
 };