I figured it was better to not dirty up the public header with a macro, but I suppose I could undef it at the end. Not a huge deal either way?
On Mon, Sep 29, 2008 at 8:17 PM, Christian Plesner Hansen <[EMAIL PROTECTED]> wrote: > Lgtm. You could consider using the standard macro instead of writing > the code in each class. > > On Mon, Sep 29, 2008 at 9:34 AM, <[EMAIL PROTECTED]> wrote: >> Reviewers: Kasper Lund, plesner, >> >> Description: >> Disallow copy and assigning a few public objects. >> >> BUG=97 >> >> Please review this at http://codereview.chromium.org/4324 >> >> Affected files: >> M include/v8.h >> >> >> Index: include/v8.h >> diff --git a/include/v8.h b/include/v8.h >> index >> fd6d9ab2783e5f07e8b5c681c8d139fce93a6a93..2842c328dc53c6aef0f167dd58b18bee9a84f4bb >> 100644 >> --- a/include/v8.h >> +++ b/include/v8.h >> @@ -783,6 +783,7 @@ class EXPORT String : public Primitive { >> protected: >> ExternalStringResource() {} >> private: >> + // Disallow copying and assigning. >> ExternalStringResource(const ExternalStringResource&); >> void operator=(const ExternalStringResource&); >> }; >> @@ -812,6 +813,7 @@ class EXPORT String : public Primitive { >> protected: >> ExternalAsciiStringResource() {} >> private: >> + // Disallow copying and assigning. >> ExternalAsciiStringResource(const ExternalAsciiStringResource&); >> void operator=(const ExternalAsciiStringResource&); >> }; >> @@ -886,6 +888,10 @@ class EXPORT String : public Primitive { >> private: >> char* str_; >> int length_; >> + >> + // Disallow copying and assigning. >> + Utf8Value(const Utf8Value&); >> + void operator=(const Utf8Value&); >> }; >> >> /** >> @@ -901,6 +907,10 @@ class EXPORT String : public Primitive { >> private: >> char* str_; >> int length_; >> + >> + // Disallow copying and assigning. >> + AsciiValue(const AsciiValue&); >> + void operator=(const AsciiValue&); >> }; >> >> /** >> @@ -915,6 +925,10 @@ class EXPORT String : public Primitive { >> private: >> uint16_t* str_; >> int length_; >> + >> + // Disallow copying and assigning. >> + Value(const Value&); >> + void operator=(const Value&); >> }; >> }; >> >> @@ -1679,6 +1693,10 @@ class EXPORT Extension { // NOLINT >> int dep_count_; >> const char** deps_; >> bool auto_enable_; >> + >> + // Disallow copying and assigning. >> + Extension(const Extension&); >> + void operator=(const Extension&); >> }; >> >> >> @@ -2199,6 +2217,10 @@ class EXPORT Locker { >> private: >> bool has_lock_; >> bool top_level_; >> + >> + // Disallow copying and assigning. >> + Locker(const Locker&); >> + void operator=(const Locker&); >> }; >> >> >> >> >> > --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---