diff options
author | Jamie Madill <jmadill@chromium.org> | 2018-10-10 16:13:02 -0400 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2018-10-10 20:39:33 +0000 |
commit | 1c7f08c3e21de8566392959b9c4dacbfc74f9acb (patch) | |
tree | 5f38a371c89ea2dba80f13136a64a1dc64f11c6f /src/libANGLE/Image.cpp | |
parent | f955bdac1ce66ace8c31499dbcb17877c60775ee (diff) | |
download | angle-1c7f08c3e21de8566392959b9c4dacbfc74f9acb.tar.gz |
Inline RefCountObject::release.
Also don't return errors from the object release methods. Not returning
errors reduces the amount of code generated. Also we shouldn't be
exiting early from destructor type functions.
Increases object binding performance.
Bug: angleproject:2877
Change-Id: Ieb8120d885b946915e355419825e1f52f31d7b49
Reviewed-on: https://chromium-review.googlesource.com/c/1270218
Reviewed-by: Geoff Lang <geofflang@chromium.org>
Commit-Queue: Jamie Madill <jmadill@chromium.org>
Diffstat (limited to 'src/libANGLE/Image.cpp')
-rw-r--r-- | src/libANGLE/Image.cpp | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/src/libANGLE/Image.cpp b/src/libANGLE/Image.cpp index 7a86733483..2753f2f245 100644 --- a/src/libANGLE/Image.cpp +++ b/src/libANGLE/Image.cpp @@ -162,7 +162,7 @@ Image::Image(rx::EGLImplFactory *factory, mState.source->addImageSource(this); } -Error Image::onDestroy(const Display *display) +void Image::onDestroy(const Display *display) { // All targets should hold a ref to the egl image and it should not be deleted until there are // no siblings left. @@ -174,8 +174,6 @@ Error Image::onDestroy(const Display *display) mState.source->removeImageSource(this); mState.source = nullptr; } - - return NoError(); } Image::~Image() |