Revert "Fix return error."
This reverts commit 9ce7bfa352e9338c2fb2c627be4d6d902a441654.
Reason for revert: should check whether the allocated buffer count is the same as requested buffer count, rather than not equals zero. This is because there is a use case when bufferhub queue is created without allocated buffers.
Bug: 115573986
Change-Id: I5a0dc67e6631248a019f3af95b801e2bb7f6d6be
diff --git a/libs/vr/libbufferhubqueue/buffer_hub_queue_client.cpp b/libs/vr/libbufferhubqueue/buffer_hub_queue_client.cpp
index e12b5a9..1f2c517 100644
--- a/libs/vr/libbufferhubqueue/buffer_hub_queue_client.cpp
+++ b/libs/vr/libbufferhubqueue/buffer_hub_queue_client.cpp
@@ -484,7 +484,7 @@
if (buffer_slots.size() == 0) {
// Error out if no buffer is allocated and improted.
ALOGE_IF(TRACE, "ProducerQueue::AllocateBuffers: no buffer allocated.");
- return ErrorStatus(ENOMEM);
+ ErrorStatus(ENOMEM);
}
return {std::move(buffer_slots)};
@@ -505,7 +505,7 @@
if (status.get().size() == 0) {
ALOGE_IF(TRACE, "ProducerQueue::AllocateBuffer: no buffer allocated.");
- return ErrorStatus(ENOMEM);
+ ErrorStatus(ENOMEM);
}
return {status.get()[0]};