ソースを参照

sm8450-common: gps: Fix -Wbitwise-instead-of-logical

Bug: http://b/215753485

Reported by clang-r445002

error: use of bitwise '|' with boolean operands [-Werro
r,-Wbitwise-instead-of-logical]

  if (gnssCbIface != nullptr || gnssCbIface_2_0 != nullptr| gnssCbIface_2_1 != nullptr) {
                                                          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
                                                          ||

Test: build with clang-r445002
Change-Id: I9410eb5208227dd70bd036aee02fb2aeb912f266
Merged-In: I9410eb5208227dd70bd036aee02fb2aeb912f266
Pirama Arumuga Nainar 3 年 前
コミット
8ab15e722f
1 ファイル変更1 行追加1 行削除
  1. 1 1
      gps/android/2.1/location_api/GnssAPIClient.cpp

+ 1 - 1
gps/android/2.1/location_api/GnssAPIClient.cpp

@@ -681,7 +681,7 @@ void GnssAPIClient::onGnssNmeaCb(GnssNmeaNotification gnssNmeaNotification)
     auto gnssCbIface_2_1(mGnssCbIface_2_1);
     mMutex.unlock();
 
-    if (gnssCbIface != nullptr || gnssCbIface_2_0 != nullptr| gnssCbIface_2_1 != nullptr) {
+    if (gnssCbIface != nullptr || gnssCbIface_2_0 != nullptr || gnssCbIface_2_1 != nullptr) {
         const std::string s(gnssNmeaNotification.nmea);
         std::stringstream ss(s);
         std::string each;