diff options
author | 2025-02-28 15:01:32 -0800 | |
---|---|---|
committer | 2025-02-28 15:01:32 -0800 | |
commit | 77a1850f6c2c432e227cd9f088b7eb989f16f35b (patch) | |
tree | 866aa58cf14698920008d1eab9ec1754e3eb4a9c /native/android/thermal.cpp | |
parent | 74da3685e232c6d02f124fdd27d2893b9aff87f1 (diff) | |
parent | 9aa43de1af68ee2b00e2b1446e25ad50d5c87750 (diff) |
Merge "Add error logging in destructor" into main
Diffstat (limited to 'native/android/thermal.cpp')
-rw-r--r-- | native/android/thermal.cpp | 14 |
1 files changed, 12 insertions, 2 deletions
diff --git a/native/android/thermal.cpp b/native/android/thermal.cpp index cefcaf7766bb..93e6ed814c34 100644 --- a/native/android/thermal.cpp +++ b/native/android/thermal.cpp @@ -139,7 +139,12 @@ AThermalManager::~AThermalManager() { mStatusListeners.clear(); if (mServiceStatusListener != nullptr) { bool success = false; - mThermalSvc->unregisterThermalStatusListener(mServiceStatusListener, &success); + auto ret = + mThermalSvc->unregisterThermalStatusListener(mServiceStatusListener, &success); + if (!success || !ret.isOk()) { + ALOGE("Failed in unregisterThermalStatusListener when AThermalManager is being " + "destroyed %d", success); + } mServiceStatusListener = nullptr; } } @@ -148,7 +153,12 @@ AThermalManager::~AThermalManager() { mHeadroomListeners.clear(); if (mServiceHeadroomListener != nullptr) { bool success = false; - mThermalSvc->unregisterThermalHeadroomListener(mServiceHeadroomListener, &success); + auto ret = mThermalSvc->unregisterThermalHeadroomListener(mServiceHeadroomListener, + &success); + if (!success || !ret.isOk()) { + ALOGE("Failed in unregisterThermalHeadroomListener when AThermalManager is being " + "destroyed %d", success); + } mServiceHeadroomListener = nullptr; } } |