From 0c9a1ed91f8e19887ac43eff5af16e59878c8226 Mon Sep 17 00:00:00 2001 From: Dan Stoza Date: Thu, 6 Apr 2017 15:10:21 -0700 Subject: libgui: Normalize IGraphicBufferConsumer methods This change modifies IGraphicBufferConsumer such that all synchronous methods return a status_t and, as such, all outputs are returned by output parameter rather than return value. It also eliminates the use of a const char* parameter that SafeInterface can't handle. Test: m -j + libgui_tests + manual testing Change-Id: I9bb0e2892492d5e0a3f660bf228834b9ed014539 --- libs/gui/ConsumerBase.cpp | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) (limited to 'libs/gui/ConsumerBase.cpp') diff --git a/libs/gui/ConsumerBase.cpp b/libs/gui/ConsumerBase.cpp index d4e4dc3616..1783561928 100644 --- a/libs/gui/ConsumerBase.cpp +++ b/libs/gui/ConsumerBase.cpp @@ -270,7 +270,9 @@ void ConsumerBase::dumpLocked(String8& result, const char* prefix) const { result.appendFormat("%smAbandoned=%d\n", prefix, int(mAbandoned)); if (!mAbandoned) { - mConsumer->dumpState(result, prefix); + String8 consumerState; + mConsumer->dumpState(String8(prefix), &consumerState); + result.append(consumerState); } } -- cgit v1.2.3-59-g8ed1b