Skip to content

Commit 344e14d

Browse files
committed
Change asserts back to regular asserts, but use the variable outside.
Some compilers are complaining about unused variables because of how assert works, and FIREBASE_ASSERT is more for externally facing things, than internal ones, so just use the variable in a basic way. PiperOrigin-RevId: 283432132
1 parent 4983f37 commit 344e14d

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

admob/src/android/banner_view_internal_android.cc

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -187,7 +187,8 @@ BoundingBox BannerViewInternalAndroid::GetBoundingBox() const {
187187
// The bounding box array must consist of 4 integers: width, height,
188188
// x-coordinate, and y-coordinate.
189189
int count = static_cast<int>(env->GetArrayLength(jni_int_array));
190-
FIREBASE_ASSERT(count == 4);
190+
assert(count == 4);
191+
(void)count;
191192

192193
jint* bounding_box_elements =
193194
env->GetIntArrayElements(jni_int_array, nullptr);

admob/src/android/native_express_ad_view_internal_android.cc

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -197,7 +197,8 @@ BoundingBox NativeExpressAdViewInternalAndroid::GetBoundingBox() const {
197197
// The bounding box array must consist of 4 integers: width, height,
198198
// x-coordinate, and y-coordinate.
199199
int count = static_cast<int>(env->GetArrayLength(jni_int_array));
200-
FIREBASE_ASSERT(count == 4);
200+
assert(count == 4);
201+
(void)count;
201202

202203
jint* bounding_box_elements =
203204
env->GetIntArrayElements(jni_int_array, nullptr);

0 commit comments

Comments
 (0)