Skip to content

Commit 1051bc5

Browse files
committed
Convert some asserts to FIREBASE_ASSERTs in admob
Some compilers are complaining about unused variables because of how assert works, so swap those to FIREBASE_ASSERT PiperOrigin-RevId: 282464729
1 parent c13e40e commit 1051bc5

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

admob/src/android/banner_view_internal_android.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -187,7 +187,7 @@ 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-
assert(count == 4);
190+
FIREBASE_ASSERT(count == 4);
191191

192192
jint* bounding_box_elements =
193193
env->GetIntArrayElements(jni_int_array, nullptr);

admob/src/android/native_express_ad_view_internal_android.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -197,7 +197,7 @@ 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-
assert(count == 4);
200+
FIREBASE_ASSERT(count == 4);
201201

202202
jint* bounding_box_elements =
203203
env->GetIntArrayElements(jni_int_array, nullptr);

0 commit comments

Comments
 (0)