Chromium Code Reviews| Index: include/v8.h |
| =================================================================== |
| --- include/v8.h (revision 2127) |
| +++ include/v8.h (working copy) |
| @@ -212,9 +212,13 @@ |
| */ |
| bool IsEmpty() const { return val_ == 0; } |
| - T* operator->() const; |
| + T* operator->() const { |
|
Kasper Lund
2009/06/11 14:51:30
I would write this on a single line just like the
antonm
2009/06/11 19:40:47
Done.
|
| + return val_; |
| + } |
| - T* operator*() const; |
| + T* operator*() const { |
|
Kasper Lund
2009/06/11 14:51:30
Ditto.
antonm
2009/06/11 19:40:47
Done.
|
| + return val_; |
| + } |
| /** |
| * Sets the handle to be empty. IsEmpty() will then return true. |
| @@ -2509,18 +2513,6 @@ |
| V8::ClearWeak(reinterpret_cast<void**>(**this)); |
| } |
| -template <class T> |
| -T* Handle<T>::operator->() const { |
| - return val_; |
| -} |
| - |
| - |
| -template <class T> |
| -T* Handle<T>::operator*() const { |
| - return val_; |
| -} |
| - |
| - |
| Local<Value> Arguments::operator[](int i) const { |
| if (i < 0 || length_ <= i) return Local<Value>(*Undefined()); |
| return Local<Value>(reinterpret_cast<Value*>(values_ - i)); |