Make all inheritance from angle::NonCopyable private so the compiler
complains about this (admittedly unlikely) code:
class Foo: angle::NonCopyable {
virtual ~Foo() { ... }
};
angle::NonCopyable *p = new Foo;
delete p;
In the above code ~Foo() is not called, only ~NonCopyable(), because the
latter is not virtual. Making it virtual would add overhead to all derived
classes which don't already have a virtual method.
Also tighten access in NonCopyable, because we can.
BUG=angleproject:2026
Change-Id: Id0dc4d959cfb7bb82cf49382118129abb1d3a4f0
Reviewed-on: https://chromium-review.googlesource.com/495352Reviewed-by:
Jamie Madill <jmadill@chromium.org>
Reviewed-by:
Corentin Wallez <cwallez@chromium.org>
Commit-Queue: Frank Henigman <fjhenigman@chromium.org>
| Name |
Last commit
|
Last update |
|---|---|---|
| .. | ||
| SystemInfo.cpp | Loading commit data... | |
| SystemInfo.h | Loading commit data... | |
| SystemInfo_internal.h | Loading commit data... | |
| SystemInfo_libpci.cpp | Loading commit data... | |
| SystemInfo_linux.cpp | Loading commit data... | |
| SystemInfo_mac.mm | Loading commit data... | |
| SystemInfo_unittest.cpp | Loading commit data... | |
| SystemInfo_win.cpp | Loading commit data... | |
| SystemInfo_x11.cpp | Loading commit data... |