diff --git a/common/General.h b/common/General.h index 7e69236999..c0aa8b58c0 100644 --- a/common/General.h +++ b/common/General.h @@ -119,8 +119,6 @@ static __fi PageProtectionMode PageAccess_Any() // -------------------------------------------------------------------------------------- // HostSys // -------------------------------------------------------------------------------------- -// (this namespace name sucks, and is a throw-back to an older attempt to make things cross -// platform prior to wxWidgets .. it should prolly be removed -- air) namespace HostSys { // Maps a block of memory for use as a recompiled code buffer. diff --git a/common/Pcsx2Defs.h b/common/Pcsx2Defs.h index 1d0eed3c98..214e01cd34 100644 --- a/common/Pcsx2Defs.h +++ b/common/Pcsx2Defs.h @@ -203,8 +203,7 @@ static constexpr size_t __pagemask = PCSX2_PAGESIZE - 1; #define safe_delete_array(ptr) \ ((void)(delete[](ptr)), (ptr) = NULL) -// No checks for NULL -- wxWidgets says it's safe to skip NULL checks and it runs on -// just about every compiler and libc implementation of any recentness. +// No checks for NULL. #define safe_free(ptr) \ ((void)(free(ptr), !!0), (ptr) = NULL) //((void) (( ( (ptr) != NULL ) && (free( ptr ), !!0) ), (ptr) = NULL)) @@ -282,23 +281,7 @@ static const pxEnumEnd_t pxEnumEnd = {}; // -------------------------------------------------------------------------------------- // This macro provides an easy and clean method for ensuring objects are not copyable. // Simply add the macro to the head or tail of your class declaration, and attempts to -// copy the class will give you a moderately obtuse compiler error that will have you -// scratching your head for 20 minutes. -// -// (... but that's probably better than having a weird invalid object copy having you -// scratch your head for a day). -// -// Programmer's notes: -// * We intentionally do NOT provide implementations for these methods, which should -// never be referenced anyway. - -// * I've opted for macro form over multi-inherited class form (Boost style), because -// the errors generated by the macro are considerably less voodoo. The Boost-style -// The macro reports the exact class that causes the copy failure, while Boost's class -// approach just reports an error in whatever "NoncopyableObject" is inherited. -// -// * This macro is the same as wxWidgets' DECLARE_NO_COPY_CLASS macro. This one is free -// of wx dependencies though, and has a nicer typeset. :) +// copy the class will give you a moderately obtuse compiler error. // #ifndef DeclareNoncopyableObject #define DeclareNoncopyableObject(classname) \ diff --git a/pcsx2/CDVD/IsoFS/IsoFS.cpp b/pcsx2/CDVD/IsoFS/IsoFS.cpp index 23ed40600d..109dafaae1 100644 --- a/pcsx2/CDVD/IsoFS/IsoFS.cpp +++ b/pcsx2/CDVD/IsoFS/IsoFS.cpp @@ -178,7 +178,7 @@ IsoFileDescriptor IsoDirectory::FindFile(const std::string_view& filePath) const if (filePath.empty()) throw Exception::FileNotFound(); - // wxWidgets DOS-style parser should work fine for ISO 9660 path names. Only practical difference + // DOS-style parser should work fine for ISO 9660 path names. Only practical difference // is case sensitivity, and that won't matter for path splitting. std::vector parts(Path::SplitWindowsPath(filePath)); IsoFileDescriptor info;