Improve Win32Handle::IsValid

- Try making the intent more clearer by making Win32Handle::IsValid virtual and override where applicable
- Also check the HANDLE in Win32File, not just the OVL

HackLib/Win32File.cpp

@@ -33,8 +33,19 @@ Win32File::Win32File(const std::filesystem::path& path, DWORD access) :
 	_size = size.LowPart;
 }
 
-Win32File::~Win32File()
+HANDLE Win32File::Value() const
 {
+	return _handle;
+}
+
+size_t Win32File::Size() const
+{
+	return _size;
+}
+
+bool Win32File::IsValid() const
+{
+	return Win32Handle::IsValid() && _ovl.Offset <= _size;
 }
 
 size_t Win32File::Read(void* buffer, size_t size)

HackLib/Win32File.hpp

@@ -7,24 +7,12 @@ class Win32File : public Win32Handle
 {
 public:
 	explicit Win32File(const std::filesystem::path& path, DWORD access = GENERIC_READ);
-	virtual ~Win32File();
 
 	NonCopyable(Win32File);
 
-	inline HANDLE Value() const
-	{
-		return _handle;
-	}
-
-	inline size_t Size() const
-	{
-		return _size;
-	}
-
-	inline operator bool() const
-	{
-		return _ovl.Offset <= _size;
-	}
+	HANDLE Value() const;
+	size_t Size() const;
+	bool IsValid() const override;
 
 	size_t Read(void* buffer, size_t size);
 	size_t ReadAt(void* buffer, size_t size, size_t offset);

HackLib/Win32Handle.hpp

@@ -13,7 +13,7 @@ class Win32Handle
 	Win32Handle& operator = (const Win32Handle& other) = delete;
 	Win32Handle& operator = (Win32Handle&& other);
 
-	bool IsValid() const;
+	virtual bool IsValid() const;
 	operator bool() const;
 
 	void Reset(HANDLE handle = nullptr);