Merge "libbinder_ndk: avoid need to allocate heap status" (9b1e10db) · Commits · e / os / android_frameworks_native-old

libs/binder/ndk/include_cpp/android/binder_auto_utils.h

+3 −0

Original line number	Diff line number	Diff line
		@@ -199,6 +199,9 @@ class ScopedAStatus : public impl::ScopedAResource<AStatus*, void, AStatus_delet
		public:
		/**
		* Takes ownership of a.
		*
		* WARNING: this constructor is only expected to be used when reading a
		* status value. Use `ScopedAStatus::ok()` instead.
		*/
		explicit ScopedAStatus(AStatus* a = nullptr) : ScopedAResource(a) {}
		~ScopedAStatus() {}

+5 −2

Original line number	Diff line number	Diff line
		@@ -23,7 +23,8 @@ using ::android::status_t;
		using ::android::binder::Status;

		AStatus* AStatus_newOk() {
		return new AStatus();
		static AStatus status = AStatus();
		return &status;
		}

		AStatus* AStatus_fromExceptionCode(binder_exception_t exception) {
		@@ -78,8 +79,10 @@ void AStatus_deleteDescription(const char* description) {
		}

		void AStatus_delete(AStatus* status) {
		if (status != AStatus_newOk()) {
		delete status;
		}
		}

		binder_status_t PruneStatusT(status_t status) {
		switch (status) {