State: Provide reference to image unit binding vector

This allows iterating through the entries without creating a copy of the vector. This was causing an assertion in FrameCapture when deconstructing the copy, which we don't need. RefCountObject.h (~BindingPointer): ! Assert failed in ~BindingPointer (src/libANGLE/RefCountObject.h:93): mObject == nullptr It also probably matches the intent of the function. Test: Aztec Ruins MEC Bug: angleproject:4048 Change-Id: I9588af3859b4d3edb5859530def1bb3992530403 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/2622610Reviewed-by: Jamie Madill <jmadill@chromium.org> Commit-Queue: Cody Northrop <cnorthrop@google.com>

State: Provide reference to image unit binding vector
2ed9671a · Cody Northrop · Commit Bot · 2e9161e4 · 2ed9671a
Commit 2ed9671a authored Jan 12, 2021 by Cody Northrop Committed by Commit Bot Jan 12, 2021
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

State.h src/libANGLE/State.h +1 -1

No files found.
--- a/src/libANGLE/State.h
+++ b/src/libANGLE/State.h
@@ -854,7 +854,7 @@ class State : angle::NonCopyable
        return mBlendFuncConstantColorDrawBuffers;
    }

-    const std::vector<ImageUnit> getImageUnits() const { return mImageUnits; }
+    const std::vector<ImageUnit> &getImageUnits() const { return mImageUnits; }

  private:
    friend class Context;