diff options
author | kwolekr <kwolekr@minetest.net> | 2015-10-27 02:51:43 -0400 |
---|---|---|
committer | kwolekr <kwolekr@minetest.net> | 2015-10-27 22:05:08 -0400 |
commit | c56d7fe0eba7905b0a63c4a1cfe909988653c23d (patch) | |
tree | fd5158b3efe0a2f2b48c54afc73f7bcecb1cb9b7 /src/basicmacros.h | |
parent | ca8e56c15a26bc5f3d1dffe5fd39e1ca4b82d6f8 (diff) | |
download | minetest-c56d7fe0eba7905b0a63c4a1cfe909988653c23d.tar.gz minetest-c56d7fe0eba7905b0a63c4a1cfe909988653c23d.tar.bz2 minetest-c56d7fe0eba7905b0a63c4a1cfe909988653c23d.zip |
Add DISABLE_CLASS_COPY macro (and use it)
Use this macro to disallow copying of an object using the assignment
operator or copy constructor. This catches otherwise silent-but-deadly
mistakes such as "ServerMap map = env->getMap();" at compile time.
If so desired, it is still possible to copy a class, but it now requires
an explicit call to memcpy or std::copy.
Diffstat (limited to 'src/basicmacros.h')
-rw-r--r-- | src/basicmacros.h | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/src/basicmacros.h b/src/basicmacros.h index 05987e32f..cebf06043 100644 --- a/src/basicmacros.h +++ b/src/basicmacros.h @@ -30,4 +30,12 @@ with this program; if not, write to the Free Software Foundation, Inc., #define CONTAINS(c, v) (std::find((c).begin(), (c).end(), (v)) != (c).end()) +// To disable copy constructors and assignment operations for some class +// 'Foobar', add the macro DISABLE_CLASS_COPY(Foobar) as a private member. +// Note this also disables copying for any classes derived from 'Foobar' as well +// as classes having a 'Foobar' member. +#define DISABLE_CLASS_COPY(C) \ + C(const C &); \ + C &operator=(const C &) + #endif |