Skip to content

Commit 6068798

Browse files
authored
Use core implementation of aggregate query equality operator. (#1376)
1 parent 6c13151 commit 6068798

File tree

2 files changed

+2
-6
lines changed

2 files changed

+2
-6
lines changed

firestore/src/main/aggregate_query_main.cc

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -71,9 +71,7 @@ Future<AggregateQuerySnapshot> AggregateQueryInternal::Get(
7171

7272
bool operator==(const AggregateQueryInternal& lhs,
7373
const AggregateQueryInternal& rhs) {
74-
// TODO(b/276440573) - there needs to be equals operator defined on
75-
// api::AggregateQuery
76-
return lhs.aggregate_query_.query() == rhs.aggregate_query_.query();
74+
return lhs.aggregate_query_ == rhs.aggregate_query_;
7775
}
7876

7977
} // namespace firestore

firestore/src/main/aggregate_query_snapshot_main.cc

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -45,9 +45,7 @@ int64_t AggregateQuerySnapshotInternal::count() const { return count_result_; }
4545

4646
bool operator==(const AggregateQuerySnapshotInternal& lhs,
4747
const AggregateQuerySnapshotInternal& rhs) {
48-
// TODO(b/276440573) - there needs to be equals operator defined on
49-
// api::AggregateQuery
50-
return lhs.aggregate_query_.query() == rhs.aggregate_query_.query() &&
48+
return lhs.aggregate_query_ == rhs.aggregate_query_ &&
5149
lhs.count_result_ == rhs.count_result_;
5250
}
5351

0 commit comments

Comments
 (0)